-
Notifications
You must be signed in to change notification settings - Fork 2.5k
Commit
- Loading branch information
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -2,5 +2,5 @@ | |
"packages": [ | ||
"packages/*" | ||
], | ||
"version": "4.9.2" | ||
"version": "4.9.1" | ||
} |
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
3a4a27f
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.
@mportuga why did you reverts commit ?
3a4a27f
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.
Because this was not an actual release that made it to mom or bower
3a4a27f
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.
it was a full release with fixes cf7a5bc 685d3c7 daaa3cb
3a4a27f
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.
It did not make it to mom, therefore not a valid release. And you don't have the repository permission to make releases
3a4a27f
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.
Deploy was performed according to the instructions and I do not need rights for the release. all deploy keys are stored in CI.
You can also make sure that the latest version of the package is on CDN https://cdn.jsdelivr.net/gh/angular-ui/bower-ui-grid/ui-grid.min.js
What does it mean
mom
?3a4a27f
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.
Well, part of the release yes, but you miss out on NPM, one of our bigger sources of users, in the future, please let me or one of the other maintainers that have access to publish on NPM handle the release
3a4a27f
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.
Could you reverser your commits and publish changes to the NPM?
Gladly if the reaction of other maintainers is more reactive.
Could you provide me with the same access to NPM to be able to timely publish critical changes ?. The reaction of the maintainers is critically low
3a4a27f
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.
@mportuga The PRs are awaiting a resolution due to the ambiguity with this issue. Can you tell me something on my part is required to speed up the process on this issue?
3a4a27f
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.
Unfortunately, i can't give you permissions at this moment, i plan on making this release over the weekend. However, i will look into seeing if I can at least give you a little bit more access, such as reviewer privelages. I just can't provide you with release privelages.
3a4a27f
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.
At the moment I have all the necessary rights in the repository, all that prevents me from making a full-fledged release is the right to publish in the npm