Skip to content

Add slot name option#25

Merged
sbq-clements merged 2 commits intomasterfrom
add-slot-name
Mar 21, 2023
Merged

Add slot name option#25
sbq-clements merged 2 commits intomasterfrom
add-slot-name

Conversation

@sbq-clements
Copy link
Copy Markdown
Contributor

@sbq-clements sbq-clements commented Feb 28, 2023

Two changes:

  • Now uses v2 instead of v1 API. It had been disabled for a while now...
  • Adds a --slot-name option to the upload trace command to be able to choose the name of the (existing or not) slot the trace will be uploaded to.

@sbq-clements sbq-clements self-assigned this Feb 28, 2023
@sbq-clements sbq-clements force-pushed the add-slot-name branch 11 times, most recently from 9a545d9 to 64b06f8 Compare March 1, 2023 10:18
@sbq-clements sbq-clements force-pushed the add-slot-name branch 6 times, most recently from 2f5e8fc to 6d01ad8 Compare March 16, 2023 08:06
Copy link
Copy Markdown

@khadijafen9-sb khadijafen9-sb left a comment

Choose a reason for hiding this comment

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

It looks good to me!

Comment thread cs_api_cli/cs_api_cli.ml Outdated
@sbq-clements sbq-clements removed the request for review from khadijafen9 March 21, 2023 14:56
Copy link
Copy Markdown

@dromaludaire dromaludaire left a comment

Choose a reason for hiding this comment

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

Looks good to me, thanks for your work!

Sorry I didn't review this earlier, I did not link my sbt github account to the Cryptosense org and totally missed that.

@sbq-clements sbq-clements merged commit aeffc11 into master Mar 21, 2023
@sbq-clements sbq-clements deleted the add-slot-name branch March 21, 2023 17:21
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