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

fix: g2p main is about to be 2.0, but we still want 1.1 on Heroku #187

Merged
merged 1 commit into from
Sep 11, 2023

Conversation

joanise
Copy link
Member

@joanise joanise commented Sep 11, 2023

We had changed things so Heroku builds always took origin/main for g2p, but now that we're working on g2p 2.0 with breaking changes, we have to revert this change and get the latest g2p 1.1.* release instead.

Fixes #186

We had changed things so Heroku builds always took origin/main for g2p,
but now that we're working on g2p 2.0 with breaking changes, we have to
revert this change and get the latest g2p 1.1.* release instead.

Fixes #186
@codecov
Copy link

codecov bot commented Sep 11, 2023

Codecov Report

Merging #187 (426ca3e) into main (9a526c2) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #187   +/-   ##
=======================================
  Coverage   87.03%   87.03%           
=======================================
  Files          21       21           
  Lines        1735     1735           
  Branches      315      315           
=======================================
  Hits         1510     1510           
  Misses        188      188           
  Partials       37       37           

Copy link
Collaborator

@roedoejet roedoejet left a comment

Choose a reason for hiding this comment

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

thanks @joanise !

@roedoejet roedoejet merged commit adfe51f into main Sep 11, 2023
6 checks passed
@roedoejet roedoejet deleted the dev.issue-186 branch September 11, 2023 22:44
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.

Revert to using only published versions of g2p
2 participants