Skip to content

Conversation

@QuentinPerez
Copy link
Contributor

⚠️ Don't merge yet ⚠️

pkg/api/cache.go Outdated
Copy link
Contributor

Choose a reason for hiding this comment

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

you don't need to specify iota each time

Copy link
Contributor

Choose a reason for hiding this comment

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

Prefix the constants

Copy link
Contributor

Choose a reason for hiding this comment

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

Do you have an example of the new output with the title under the list of candidates ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

- 5f19794c          bootscript:3-2-34-apparmor-31          3.2.34-apparmor #31            arm
- 7c7fcac7          bootscript:4-2-5-apparmor-259          4.2.5-apparmor #259            arm
- 1e7a3063          bootscript:armv7l-4-3-3-std-1-latest   armv7l 4.3.3 std #1 (latest)   arm
FATA[0000] cannot execute 'run': failed to create server: Too many candidates for m (3)

Copy link
Contributor

Choose a reason for hiding this comment

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

perfect

pkg/api/api.go Outdated
Copy link
Contributor

Choose a reason for hiding this comment

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

I think that you will soon need to add the region here, so maybe we should switch to a structure

@moul
Copy link
Contributor

moul commented Jan 7, 2016

LGTM, you can fix my comments now or later as they are clearly not important

We need to test this version for a while so we can adjust the details, so as soon as the PR is merged, I will create a pre-release

QuentinPerez pushed a commit that referenced this pull request Jan 8, 2016
Add region and arch field for images
@QuentinPerez QuentinPerez merged commit bcf357d into scaleway:master Jan 8, 2016
clement-gilbert pushed a commit to clement-gilbert/scaleway-cli that referenced this pull request Mar 3, 2022
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