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

Setup Tests and Modernize #1

Merged
merged 2 commits into from
Oct 21, 2023
Merged

Setup Tests and Modernize #1

merged 2 commits into from
Oct 21, 2023

Conversation

confused-Techie
Copy link
Member

This PR adds GitHub Actions based testing to this repository, as well as updates some of the information within the package.json to be up to date and reflect the package's current status.

Copy link
Member

@DeeDeeG DeeDeeG left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is neat!

Good to have the tests automatically running. (Not that there are very many tests to run, apparently!)

And the changes to package.json look good.

Thanks for doing this!

P.S. we should be able to test on newer than NodeJS 16 if we switch superstring to the copy at pulsar-edit/superstring repo.

The fact that this package depends on Atom's outdated superstring, and not pulsar-edit's copy that can build on Node newer than 16, is also affecting github package, which depends on whats-my-line and therefore indirectly on Atom's outdated copy of superstring...

@DeeDeeG DeeDeeG merged commit fa53d7c into master Oct 21, 2023
1 check passed
@confused-Techie confused-Techie deleted the setup-tests branch October 23, 2023 06:10
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