Skip to content

Conversation

rybit
Copy link
Member

@rybit rybit commented Oct 7, 2022

BREAKING CHANGES:

  • removes the packages:
    • metriks -> github.com/netlify/go-observability
    • tracing -> github.com/netlify/go-observability
    • utils -> github.com/netlify/go-utils
    • testutils -> github.com/netlify/go-test-utils
    • server -> github.com/netlify/go-http
    • router -> github.com/netlify/go-http
    • featureflag -> github.com/netlify/go-flags
    • nconf -> github.com/netlify/go-config

Then in each service we'd be able to configure those utilities in the service to allow for independent versioning per service. No more huge upgrades!

I've copied the go-http/ntls structs to individual packages here so that they can be independently deprecated or ported. They're repeated files but I thought that better to be able to use the separately.

Note: these are all private packages, we need to make sure that you've got CI setup to access them properly

Examples:

@rybit rybit requested a review from a team as a code owner October 7, 2022 22:45
@@ -0,0 +1,100 @@
package mongoclient
Copy link
Member Author

Choose a reason for hiding this comment

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

this is stolen from go-http/ntls

@rybit rybit changed the title remove packages that have been broken out feat!: remove packages that have been broken out Oct 7, 2022
@rybit rybit requested a review from a team October 7, 2022 22:52
@rybit rybit merged commit e5f9fcd into main Oct 11, 2022
@rybit rybit deleted the breakup-commons branch October 11, 2022 17:18
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