Skip to content

Conversation

mfisher87
Copy link
Member

@mfisher87 mfisher87 commented Aug 22, 2025

Description

3.0.0 was released today, and now check-release CI is failing.

Cannot find module '@fortawesome/react-fontawesome' or its corresponding type declarations.

This change should get things working again.

Checklist

  • PR has a descriptive title and content.
  • PR description contains references to any issues the PR resolves, e.g. Resolves #XXX.
  • PR has one of the labels: documentation, bug, enhancement, feature, maintenance
  • Checks are passing.
    Failing lint checks can be resolved with:
    • pre-commit run --all-files
    • jlpm run lint

📚 Documentation preview: https://jupytergis--888.org.readthedocs.build/en/888/
💡 JupyterLite preview: https://jupytergis--888.org.readthedocs.build/en/888/lite

Copy link
Contributor

Binder 👈 Launch a Binder on branch mfisher87/jupytergis/avoid-react-fontawesome-breaking-change

@mfisher87 mfisher87 requested a review from arjxn-py August 22, 2025 21:21
3.0.0 was released today, and now check-release CI is failing.

```
Cannot find module '@fortawesome/react-fontawesome' or its corresponding type declarations.
```

This change should get things working again.
Copy link
Contributor

github-actions bot commented Aug 22, 2025

Integration tests report: appsharing.space

@mfisher87 mfisher87 changed the title Avoid breaking change in react-fontawesome 3.0.0 Avoid react-fontawesome 3.0.0 for now because of breaking change Aug 22, 2025
@mfisher87 mfisher87 enabled auto-merge (squash) August 22, 2025 21:51
@mfisher87 mfisher87 mentioned this pull request Aug 23, 2025
4 tasks
@mfisher87 mfisher87 merged commit 51b7d2c into geojupyter:main Aug 23, 2025
16 checks passed
@mfisher87 mfisher87 deleted the avoid-react-fontawesome-breaking-change branch August 23, 2025 16:14
mfisher87 added a commit to mapninja/jupytergis-docwork that referenced this pull request Aug 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants