Skip to content

#7 implemented craas-go v2 tokens#8

Merged
pavelpereyma merged 8 commits intoselectel:mainfrom
Bablzz:main
Jul 22, 2025
Merged

#7 implemented craas-go v2 tokens#8
pavelpereyma merged 8 commits intoselectel:mainfrom
Bablzz:main

Conversation

@Bablzz
Copy link
Contributor

@Bablzz Bablzz commented Jun 3, 2025

For #7

  1. Add v2/tokens
  2. Separate ServiceClient in module and use it in v1 and v2(not break changes)
  3. Fix gotest.sh by shellcheck
  4. Fix run unit tests by make

@Bablzz Bablzz marked this pull request as draft June 5, 2025 12:46
@Bablzz Bablzz marked this pull request as ready for review June 6, 2025 09:44
Copy link
Contributor

@pavelpereyma pavelpereyma left a comment

Choose a reason for hiding this comment

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

Let's remove outdated/duplicate constants from file pkg/client.go.

sokolnikov.m added 2 commits July 2, 2025 17:04
@Bablzz
Copy link
Contributor Author

Bablzz commented Jul 2, 2025

Let's remove outdated/duplicate constants from file pkg/client.go.

delete from pkg/client

@Bablzz
Copy link
Contributor Author

Bablzz commented Jul 22, 2025

unit tests are ok, there is problem with - gocovmerge: command not found

@pavelpereyma pavelpereyma linked an issue Jul 22, 2025 that may be closed by this pull request
@pavelpereyma pavelpereyma merged commit e857114 into selectel:main Jul 22, 2025
1 of 4 checks 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.

Add V2 Token API support

3 participants