Skip to content

fix: Revert "chore: Start SDK semantic versioning from v1"#366

Merged
kodiakhq[bot] merged 1 commit intomainfrom
revert-362-feat/update_sdk_to_v1
Nov 7, 2022
Merged

fix: Revert "chore: Start SDK semantic versioning from v1"#366
kodiakhq[bot] merged 1 commit intomainfrom
revert-362-feat/update_sdk_to_v1

Conversation

@hermanschaaf
Copy link
Copy Markdown
Contributor

@hermanschaaf hermanschaaf commented Nov 7, 2022

Reverts #362

I think the change to use v1 is invalid, as Go expects versioning to be used only from v2 onwards. When importing it in cloudquery/cloudquery, I'm getting this error:

require github.com/cloudquery/plugin-sdk/v1: version "v1.0.1" invalid: malformed module path "github.com/cloudquery/plugin-sdk/v1"

@erezrokah
Copy link
Copy Markdown
Member

erezrokah commented Nov 7, 2022

TODO after this:
1. Delete the v1 releases https://github.com/cloudquery/plugin-sdk/releases
~~2. Delete the v2 tags https://github.com/cloudquery/plugin-sdk/tags~v
3. Remove the autorelease labels from #361 and
#365 so they don't get re-released

Or just merge this as fix:

@hermanschaaf hermanschaaf changed the title Revert "chore: Start SDK semantic versioning from v1" fix: Revert "chore: Start SDK semantic versioning from v1" Nov 7, 2022
@github-actions github-actions bot added the fix label Nov 7, 2022
@yevgenypats
Copy link
Copy Markdown
Contributor

What are the v2 tags? I dont see any.

Regarding deleting the previous ones. I think no need as no one will import them and we just bump a minor version to v1.0.2 and all should work.

starting v2 in any case we need to have /v2 and it has to match v2.x.z

@erezrokah
Copy link
Copy Markdown
Member

Regarding deleting the previous ones. I think no need as no one will import them and we just bump a minor version to v1.0.2 and all should work.

💯 Just revised my comment

@kodiakhq kodiakhq bot merged commit c66be4b into main Nov 7, 2022
@kodiakhq kodiakhq bot deleted the revert-362-feat/update_sdk_to_v1 branch November 7, 2022 16:05
kodiakhq bot pushed a commit that referenced this pull request Nov 7, 2022
🤖 I have created a release *beep* *boop*
---


## [1.0.2](v1.0.1...v1.0.2) (2022-11-07)


### Bug Fixes

* Revert "chore: Start SDK semantic versioning from v1" ([#366](#366)) ([c66be4b](c66be4b))

---
This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants