-
-
Notifications
You must be signed in to change notification settings - Fork 496
[Mercure] Fix enable_profiler node position #672
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
Conversation
chalasr
commented
Nov 8, 2019
Q | A |
---|---|
License | MIT |
Doc issue/PR | - |
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.
Pull request passes validation.
Actually |
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.
Pull request does not pass validation.
Right, fixed! |
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.
Pull request does not pass validation.
682098f
to
26c23cb
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.
Pull request does not pass validation.
The validation error is because the master branch has alias 1.0.x-dev, not 0.3.x-dev (which looks wrong to me btw, if you plan multiple 0.x releases before reaching 1.0, with BC breaks between them) |
Also, why 0.3 ? There is no 0.2 release yet |
You’re right, the next bundle version is 0.2 (I mixed with the library version). And you’re also right about the branch alias. Both the bundle and the lib must be updated. |
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.
Pull request does not pass validation.
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.
Pull request does not pass validation.
This PR can now be merged (the CI will be green now). |
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.
Pull request passes validation.
Green |