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.

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

Updated stylus dependencies to 0.54.8. #348

Merged
merged 1 commit into from
Nov 27, 2020
Merged

Conversation

AlynxZhou
Copy link
Contributor

Fix #347, remove NodeJS 14 warnings.

@stevenjoezhang
Copy link

stevenjoezhang commented May 31, 2021

@xdan @iChenLei Is it possible to publish a new version of nib on npm that includes this commit? Thanks!

@iChenLei
Copy link
Member

iChenLei commented Sep 6, 2021

2021-09-06 21 13 22

https://www.npmjs.com/package/nib

Only @tj and @notslang have the permission to publish nib package, @stevenjoezhang I am so sorry, I can't solve this problem immediately, I must send email to tj or slang first.

@stevenjoezhang
Copy link

Thanks very much for your reply. nib is an important dependency of the blog framework Hexo and is used by many blog themes. I would appreciate if the package can be updated on npm.

@notslang
Copy link
Collaborator

notslang commented Sep 7, 2021

@iChenLei - I looked though the commit log and it seems like we're just updating deps and test cases. If there's no new functionality I'll release the last 31 commits as a patch. Do you think that's good, or should this be a new minor version?

@iChenLei
Copy link
Member

iChenLei commented Sep 8, 2021

@notslang #350 I create a pr for ci, replace outdated travis-ci with github actions, and make stylus as peerDependencies and not limit version, semver is wildcard *. please review. I think minor version is better v1.2.0, and please help me add this big change in changelog, this is important for nib user. thanks

https://github.com/iChenLei/nib/actions/runs/1211720732 New CI is work as expected.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Please upgrade stylus dependency version to fix NodeJS 14 warnings
5 participants