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

Change gpa to score to better reflect variance in international scoring systems #391

Merged
merged 1 commit into from
Aug 28, 2020

Conversation

DonDebonair
Copy link
Member

Fixes #288, supersedes #328

@DonDebonair
Copy link
Member Author

@stp-ip @thomasdavis can you have a look? I'd like to merge this asap.

@stp-ip
Copy link
Member

stp-ip commented Aug 25, 2020

Codewise this looks good. Still not sure we should be merging in breaking changes until we know what our overall direction is. If we agree, that the v1 branch is breaking and currently open for development and open to future breaking changes until we do a release, I'm happy to get this in now and discuss other options later :)

@thomasdavis
Copy link
Member

LGTM.

We will have some breaking changes, but minimal.

@DonDebonair
Copy link
Member Author

@stp-ip @thomasdavis as far as I'm concerned we can integrate breaking changes into the v1.0.0 branch. In the README we state that we adhere to the Semantic Versioning standard, and given that we haven't done an official (tagged) v1.0 release yet, breaking changes are fair game.

@thomasdavis
Copy link
Member

@dandydev Very true. We also will be only affecting thousands, not millions, let's bear the cost for now.

@DonDebonair DonDebonair merged commit 604fb34 into jsonresume:v1.0.0 Aug 28, 2020
This was referenced Aug 28, 2020
@github-actions
Copy link

🎉 This PR is included in version 0.2.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

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.

Add Marking Scales for non-USA Courses.
3 participants