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

Pull panda-hmac into repo #124

Merged
merged 2 commits into from
Nov 21, 2023
Merged

Pull panda-hmac into repo #124

merged 2 commits into from
Nov 21, 2023

Conversation

andrew-nowak
Copy link
Member

@guardian/digital-cms

Pull the contents of the panda-hmac repo into this one, allowing us to release all panda-related modules simultaneously and reducing repo fragmentation.

@andrew-nowak andrew-nowak requested a review from a team as a code owner November 20, 2023 15:36
@@ -1 +1 @@
version in ThisBuild := "1.4.1-SNAPSHOT"
version in ThisBuild := "3.0.0-SNAPSHOT"
Copy link
Member Author

Choose a reason for hiding this comment

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

big bump but IMO, panda-hmac should be on the same version numbers as the other modules, but currently panda-hmac is on 2.2.0 and the rest of panda is on 1.4.0.
This could be 2.3.0 but it seems weird to me that most panda modules would not only jump to a new major version, but not even have a x.0.0 release? maybe i'm overthinking it though

Copy link
Contributor

@jonathonherbert jonathonherbert left a comment

Choose a reason for hiding this comment

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

👍 👍

IIUC it's only the play actions that have a dependency on panda – wonder if this repo will eventually be gu-authentication 🤔

@andrew-nowak
Copy link
Member Author

are you referring to hmac-headers? yeah I'm leaving that repo where it is since it's nicely independent of the rest of the panda ecosystem

@andrew-nowak andrew-nowak merged commit 0a27d3d into main Nov 21, 2023
1 check passed
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