Skip to content
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

Drop Node < 18, update deprecated glob via test-exclude #301

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

dmurvihill
Copy link

Fixes #300.

@dmurvihill dmurvihill changed the title feat!: Drop Node < 18, update deprecated glob via test-exclude Drop Node < 18, update deprecated glob via test-exclude Jan 8, 2025
@dmurvihill dmurvihill changed the base branch from master to next January 8, 2025 22:16
@dmurvihill dmurvihill changed the base branch from next to master January 8, 2025 22:22
@bcoe bcoe added the ci label Jan 15, 2025
@mikehaas763
Copy link

@dmurvihill I think you just gotta remove the unsupported node versions from the matrix in .github/workflows /ci.yaml. Are you available to make that change and maybe we can get this fix rolling downstream?

@dmurvihill
Copy link
Author

Thanks, fixed (I think).

Note that "release-please" also needed updating from Node 14 to 18 and so you may wish to test that as well.

@mikehaas763
Copy link

Hi @bcoe! Are you able to approve the pipeline workflow so we can see if there are any other errors?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update to test-exclude 7 to cure deprecated sub-dependency
3 participants