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

build: use release-please for automating releases #622

Merged
merged 11 commits into from Sep 13, 2021
Merged

Conversation

bcoe
Copy link
Member

@bcoe bcoe commented Sep 11, 2021

  • migrate to using release-please for builds.
  • migrated away from using babel-register for istanbul-lib-instrument, which is technically a breaking change but shouldn't affect anyone.
  • Had to update some tests due to [Bug]: @babel/parser@7.14.8 breaks IstanbulJS skip logic babel/babel#13750.
  • switched to GitHub actions for tests.
  • dropping Node 8 since it's years past EOL at this point, and tests were broken on Node 8. I opted to keep Node 8 for now, it was the mocha dependency that needed to be downgraded.

Refs: babel/babel#13750

@bcoe bcoe merged commit 35309f7 into master Sep 13, 2021
@bcoe bcoe deleted the automate-deployment branch September 13, 2021 01:44
bcoe added a commit that referenced this pull request Sep 13, 2021
istanbul-lib-instrument no longer uses babel, this changes export behavior to no longer
use .defaults, which may break some implementers.

Refs: #622
vivek-freshworks pushed a commit to freshworks/istanbuljs that referenced this pull request Oct 16, 2023
vivek-freshworks pushed a commit to freshworks/istanbuljs that referenced this pull request Oct 16, 2023
istanbul-lib-instrument no longer uses babel, this changes export behavior to no longer
use .defaults, which may break some implementers.

Refs: istanbuljs#622
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.

None yet

1 participant