-
Notifications
You must be signed in to change notification settings - Fork 31
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
generators: add --no-branch option to prevent creating new local branch (#248) #249
Conversation
0953642
to
f161a48
Compare
superflore/generators/bitbake/run.py
Outdated
@@ -93,7 +93,8 @@ def main(): | |||
overlay = RosMeta( | |||
_repo, | |||
not args.output_repository_path, | |||
branch='superflore/{}'.format(now), | |||
branch=(('superflore/{}'.format(now)) if not args.no_branch | |||
else None), |
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.
@shr-project the linter is unhappy with this line
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.
Looking into it now (before the line was too long now this.. :))
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.
before the line was too long now this.. :)
Haha, yeah. Welcome to superflore changes ;). Linter is pretty picky.
Fixes #248 |
f161a48
to
3302c1e
Compare
3302c1e
to
8f09e9a
Compare
Hi @allenh1 is there anything blocking this PR? We're almost ready to publish updated meta-ros with recipes generated with superflore modified with all these open PRs and it would be nice to have it in superflore/master before we do. |
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.
Some small changes, but nothing major.
* useful when regenerating the files locally by hand without planing to submit PR with the changes or when some other tool will do additional integration before doing the PR (like we plan to do with meta-ros) Signed-off-by: Martin Jansa <martin.jansa@lge.com>
8f09e9a
to
6aa9b53
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.
@shr-project Thanks!
Looks good.
…ch (ros-infrastructure#249) * useful when regenerating the files locally by hand without planing to submit PR with the changes or when some other tool will do additional integration before doing the PR (like we plan to do with meta-ros) Signed-off-by: Martin Jansa <martin.jansa@lge.com>
PR with the changes or when some other tool will do additional
integration before doing the PR (like we plan to do with meta-ros)
Signed-off-by: Martin Jansa martin.jansa@lge.com