-
Notifications
You must be signed in to change notification settings - Fork 8
Improvement/artesca 7968 bump font awesome v7 #998
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Improvement/artesca 7968 bump font awesome v7 #998
Conversation
Hello jeanmarcmilletscality,My role is to assist you with the merge of this Available options
Available commands
Status report is not available. |
Waiting for approvalThe following approvals are needed before I can proceed with the merge:
Peer approvals must include at least 1 approval from the following list: |
…sion for icon dynamic imports
…er than title. Deprecate title Also update IconHelper Remove table from Icon.component.tsx but keep export
Waiting for approvalThe following approvals are needed before I can proceed with the merge:
Peer approvals must include at least 1 approval from the following list: |
|
/approve |
Waiting for approvalThe following approvals are needed before I can proceed with the merge:
Peer approvals must include at least 1 approval from the following list: The following options are set: approve |
|
I have successfully merged the changeset of this pull request
Please check the status of the associated issue ARTESCA-7968. Goodbye jeanmarcmilletscality. The following options are set: approve |
Bump font awesome to v7
Keep dynamic imports but use comment to avoid importing problematic module "import.macro.js" from font awesome