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

[dalek] Bump dependencies to latest, fix links #602

Merged
merged 3 commits into from
Jun 24, 2023

Conversation

confused-Techie
Copy link
Member

Low hanging fruit PR, bumped our dependencies in dalek as well as fixing a link in the deprecation notice, that still linked to Atom's repo. Nothing to crazy, ensured tests ran just fine, and compared the diffs between these new versions to ensure there were no to crazy API changes.

Additionally updated the engines.atom version to point to our latest release.

@Spiker985
Copy link
Member

Just to be sure, the change to engine would mean that next release this wouldn't work, correct?

Just making sure that is the intended change

@confused-Techie
Copy link
Member Author

@Spiker985 Good catch there. That was not the intended change. Must have mistakenly saw the short value of the semver and assumed it was already out of date with the version we were currently using. I've changes that one back, thanks for seeing it

@Spiker985
Copy link
Member

No problem!

Otherwise, barring testing issues, should be good 👍

@confused-Techie
Copy link
Member Author

@Spiker985 I appreciate the review! Thank you!

@confused-Techie confused-Techie merged commit 6af9c14 into master Jun 24, 2023
@confused-Techie confused-Techie deleted the bump-dalek-deps branch June 24, 2023 00:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants