Skip to content
This repository has been archived by the owner on Aug 7, 2023. It is now read-only.

Upgrade to latest deps: prevent false positives on modern Coffee syntax #68

Closed
wants to merge 1 commit into from

Conversation

elektronik2k5
Copy link

Such as import/export

@@ -2,7 +2,7 @@
"name": "linter-coffeelint",
"main": "./lib/init",
"activationCommands": {},
"version": "1.1.2",
"version": "1.2.0",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Don't update the version, this will be done when releasing a new version.

"atom-package-deps": "3.0.6",
"coffeelint": "1.9.7",
"resolve": "1.1.5"
"atom-package-deps": "4.3.1",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can this be made a range please.

"resolve": "1.1.5"
"atom-package-deps": "4.3.1",
"coffeelint": "1.16.0",
"resolve": "1.1.7"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Again, this should be a range as it's a utility library.

@Arcanemagus
Copy link
Member

Very interesting... it looks like somebody disabled GreenKeeper on this repository, so none of the PRs that should have been coming in updating this have well... been coming in. 😠

@Arcanemagus
Copy link
Member

Went ahead and made the changes in #70, thanks for bringing this up!

@Arcanemagus
Copy link
Member

Published v1.2.0, now to add specs and clean up the rest of this mess.

@elektronik2k5
Copy link
Author

@Arcanemagus awesome, thanks!

@elektronik2k5 elektronik2k5 deleted the upgrade-deps branch October 27, 2016 08:41
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants