Skip to content

Roll version numbers for 1.9.4 release #1223

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

Merged
merged 1 commit into from
Sep 26, 2020
Merged

Conversation

baylesj
Copy link
Contributor

@baylesj baylesj commented Sep 26, 2020

No description provided.

@baylesj baylesj merged commit 9059f5c into master Sep 26, 2020
@baylesj baylesj deleted the roll-version-numbers branch September 26, 2020 02:19
@@ -9,7 +9,7 @@
// 3. /CMakeLists.txt
// IMPORTANT: also update the SOVERSION!!

#define JSONCPP_VERSION_STRING "1.9.3"
#define JSONCPP_VERSION_STRING "1.9.4"
#define JSONCPP_VERSION_MAJOR 1
#define JSONCPP_VERSION_MINOR 9
#define JSONCPP_VERSION_PATCH 3

Choose a reason for hiding this comment

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

updating the PATCH here was missed.

Copy link
Contributor

Choose a reason for hiding this comment

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

Also

 #define JSONCPP_VERSION_PATCH 3

@cdunn2001
Copy link
Contributor

This was merged before AppVeyor had passed. Not good!

I'm trying to figure out when TravisCI stopped running. We're lucky AppVeyor still works I think.

@cdunn2001
Copy link
Contributor

Weird. My latest PR failed in TravisCI -- so obviously it did run in TravisCI -- but there is no TravisCI notification in GitHub.

Possibly related: https://travis-ci.community/t/github-ci-notifiactions-from-travis-stopped-working/9238/3

Still investigating ...

@cdunn2001
Copy link
Contributor

Fixed it by adding a new "Branch Protection Rule". No worries.

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.

3 participants