New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Explicit channel #24
Explicit channel #24
Conversation
b0bbcf3
to
63064fd
Compare
The scopes are still used to verify permission when pushing to a real store.
63064fd
to
175daf7
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome, I like this approach! Let's add a couple of esr/candidates
cases before merging 🙂
@pytest.mark.parametrize('channel, push_to_store, expected', ( | ||
('beta', True, True), | ||
('candidate', True, True), | ||
('esr/stable', True, True), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nit: Let's add a case for esr/candidate
@pytest.mark.parametrize('raises, scopes, channel', ( | ||
(False, ['project:releng:snapcraft:firefox:candidate'], 'candidate'), | ||
(False, ['project:releng:snapcraft:firefox:beta'], 'beta'), | ||
(False, ['project:releng:snapcraft:firefox:esr'], 'esr/stable'), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nit: Let's add a case for esr/candidate
|
||
channel = scope[len(SNAP_SCOPES_PREFIX):] | ||
def get_snap_channel(config, task): | ||
if 'channel' in task['payload']: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 for preserving backwards compatibility \o/
Depends on #25 |
Stop importing snapcraft via submodule.
bf846d9
to
31eeabf
Compare
No description provided.