Skip to content

fix(live): fix usage of force-namespace parameter in live loader#7526

Merged
NamanJain8 merged 1 commit intomasterfrom
naman/live-parameter
Mar 8, 2021
Merged

fix(live): fix usage of force-namespace parameter in live loader#7526
NamanJain8 merged 1 commit intomasterfrom
naman/live-parameter

Conversation

@NamanJain8
Copy link
Copy Markdown
Contributor

@NamanJain8 NamanJain8 commented Mar 8, 2021

This PR fixes the behaviour of --force-namespace flag in live loader.
Earlier, we used to silently ignore this flag when passed by the non-galaxy user. Now, we throw an error if the force-namespace flag is used by the non-galaxy user.


This change is Reviewable

@github-actions github-actions Bot added area/live-loader Issues related to live loading. area/testing Testing related issues labels Mar 8, 2021
@NamanJain8 NamanJain8 changed the title fix(live): fix usage of force-namespace parameter in export fix(live): fix usage of force-namespace parameter in live loader Mar 8, 2021
@NamanJain8 NamanJain8 merged commit 4de8505 into master Mar 8, 2021
@NamanJain8 NamanJain8 deleted the naman/live-parameter branch March 8, 2021 13:36
aman-bansal pushed a commit that referenced this pull request Mar 9, 2021
This PR fixes the behaviour of --force-namespace flag in live loader.
Earlier, we used to silently ignore this flag when passed by the non-galaxy user. Now, we throw an error if the force-namespace flag is used by the non-galaxy user.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/live-loader Issues related to live loading. area/testing Testing related issues

Development

Successfully merging this pull request may close these issues.

2 participants