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

Play dependencies silently resolve not found dependencies #1051

Closed
xael-fry opened this issue Dec 22, 2016 · 1 comment
Closed

Play dependencies silently resolve not found dependencies #1051

xael-fry opened this issue Dec 22, 2016 · 1 comment
Assignees
@xael-fry
Milestone
1.4.4

Comments

@xael-fry
Copy link
Member

On local you a module "test" version1.16.
It deps if you set "test" 1.15, the dependencies will not resolve the module but no error is displayed.
If you use verbose parameters, you will have:

pxpLocalModules: bad revision found in E:\play-modules\test\conf\dependencies.yml: expected='1.16 found='1.16'

But no error in the console
@xael-fry xael-fry self-assigned this Dec 22, 2016
@xael-fry xael-fry added this to the 1.4.4 milestone Dec 22, 2016
xael-fry added a commit to xael-fry/play that referenced this issue Dec 23, 2016
@xael-fry
* [playframework#1051] fix(DependenciesManager): display error when r…
2f72cc7 
…unning dependencies
@xael-fry xael-fry mentioned this issue Dec 23, 2016
Merged
xael-fry added a commit to xael-fry/play that referenced this issue Dec 26, 2016
@xael-fry
* [playframework#1051] fix(DependenciesManager): display error when r…
533ce8d 
…unning dependencies
xael-fry added a commit to xael-fry/play that referenced this issue Dec 27, 2016
@xael-fry
* [playframework#1051] fix(DependenciesManager): display error when r…
a1bbc0a 
…unning dependencies
xael-fry added a commit to xael-fry/play that referenced this issue Dec 27, 2016
@xael-fry
* [playframework#1051] fix(dependencies): remove locale variable huma…
967228e 
…nReadyLogger
xael-fry added a commit that referenced this issue Dec 27, 2016
@xael-fry
Merge pull request #1054 from xael-fry/1051_playDepsErrorDisplay
98dfd40 
* [#1051] fix(DependenciesManager): display error when running dependencies
@xael-fry
Copy link
Member Author

Merged in 1.4.x

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@xael-fry xael-fry

Labels
None yet
Projects
None yet
Milestone
1.4.4
Development

No branches or pull requests
1 participant
@xael-fry