Skip to content
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.

Sign up for GitHub

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

Jump to bottom

cli: unable to run action defined in 3rd level of a repo directory #658

Closed
arshul opened this issue May 28, 2019 · 1 comment
Closed

cli: unable to run action defined in 3rd level of a repo directory #658

arshul opened this issue May 28, 2019 · 1 comment
Labels
bug

Comments

@arshul
Copy link
Contributor

arshul commented May 28, 2019

An action defined in user/repo/dir@master runs but if an action is defined as in user/repo/dir/subdir@master fails to run.
Popper is only able to run actions from up to the 2nd level directory and not more than that.
@ivotron ivotron added the bug label May 28, 2019
@ivotron ivotron changed the title Unable to run action defined in 3rd level of a repo directory. [BUG] cli: unable to run action defined in 3rd level of a repo directory May 28, 2019
ivotron added a commit that referenced this issue May 28, 2019
@ivotron
Fixes splitting of Git URLs
f91db62 
fixes #658
@ivotron ivotron mentioned this issue May 28, 2019
Closed
@ivotron
Copy link
Collaborator

ivotron commented Jun 6, 2019

this will be subsumed by #623

@ivotron ivotron closed this as completed Jun 6, 2019
@ivotron ivotron reopened this Jun 6, 2019
@JayjeetAtGithub JayjeetAtGithub mentioned this issue Jun 10, 2019
Merged
ivotron pushed a commit that referenced this issue May 24, 2020
@JayjeetAtGithub @ivotron
Rewrites scm.parse() with re (#677)
acca747 
fixes #623 
fixes #658
ivotron pushed a commit that referenced this issue May 25, 2020
@JayjeetAtGithub @ivotron
Rewrites scm.parse() with re (#677)
723b4d7 
fixes #623 
fixes #658
ivotron pushed a commit that referenced this issue May 25, 2020
@JayjeetAtGithub @ivotron
Rewrites scm.parse() with re (#677)
a5d80c4 
fixes #623 
fixes #658
ivotron pushed a commit that referenced this issue May 25, 2020
@JayjeetAtGithub @ivotron
Rewrites scm.parse() with re (#677)
218275c 
fixes #623 
fixes #658
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fixes splitting of Git URLs getpopper/popper
2 participants
@ivotron @arshul