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

greenkeeper/initial #5

Open
wants to merge 11 commits into
base: master
Choose a base branch
from
Open

greenkeeper/initial #5

wants to merge 11 commits into from

Conversation

gregswindle
Copy link
Owner

@gregswindle gregswindle commented Nov 10, 2017

Description of change

Allow Greenkeeper to manage JavaScript dependency drift with automated updates and PRs.

Associated issue(s)

Closes #1

PR check-list

✅ Please review and check the appropriate items.

1. Acceptance criteria

  • The acceptance criteria for all associated issues have been completed, tested, and validated.

2. Test coverage Coverage Status

  • The source code is 100% covered with passing specs.

3. Code standards compliance

JavaScript Style Guide Google Checkstyle Docs

3.1. JavaScript

  • ESLint passes.

3.2. Java

  • Checkstyle passes
  • PMD passes
  • Findbugs passes

4. Code quality Quality Gate

  • The quality gateways pass with an "A" grade.
Measure Scores
Complexity Complexity
Cognitive complexity
Complexity per Class
Complexity per file
Complexity per Function
Duplications Duplications
Issues Issues
Maintainability Code smells
Maintainability
Technical debt
Reliability Reliability
Security Security

ℹ️ These tasks are not required to open a PR, and may be addresses while the PR is open.

@greenkeeper
Copy link

greenkeeper bot commented Nov 10, 2017

Let’s get started with automated dependency management for maven-code-quality-pom 💪

⚠️ Greenkeeper has found a package-lock.json file in this repository. Please use greenkeeper-lockfile to make sure this gets updated as well.

This pull request updates all your dependencies to their latest version. Having them all up to date really is the best starting point for keeping up with new releases. Greenkeeper will look out for further dependency updates and make sure to handle them in isolation and in real-time, but only after you merge this pull request.

Important: Greenkeeper will only start watching this repository’s dependency updates after you merge this initial pull request.


💥 Tests on this branch are failing. Here’s how to proceed.

To solve the issue, first find out which of the dependency’s updates is causing the problem. Then fix your code to accomodate the changes in the updated dependency. next-update is a really handy tool to help you with this.

Then push your changes to this branch and merge it.

🏷 How to check the status of this repository

Greenkeeper adds a badge to your README which indicates the status of this repository.

This is what your badge looks like right now 👉 Greenkeeper badge

🙈 How to ignore certain dependencies

You may have good reasons for not wanting to update to a certain dependency right now. In this case, you can change the dependency’s version string in the package.json file back to whatever you prefer.

To make sure Greenkeeper doesn’t nag you again on the next update, add a greenkeeper.ignore field to your package.json, containing a list of dependencies you don’t want to update.

// package.json
{
  
  "greenkeeper": {
    "ignore": [
      "package-names",
      "you-want-me-to-ignore"
    ]
  }
}
👩‍💻 How to update this pull request
  # Change into your repository’s directory
  git fetch
  git checkout greenkeeper/initial
  npm install-test
  # Adapt your code until everything works again
  git commit -m 'chore: adapt code to updated dependencies'
  git push origin greenkeeper/initial
✨ How do dependency updates work with Greenkeeper?

After you merge this pull request, Greenkeeper will create a new branch whenever a dependency is updated, with the new version applied. The branch creation should trigger your testing services and check whether your code still works with the new dependency version. Depending on the the results of these tests Greenkeeper will try to open meaningful and helpful pull requests and issues, so your dependencies remain working and up-to-date.

-  "underscore": "^1.6.0"
+  "underscore": "^1.7.0"

The above example shows an in-range update. 1.7.0 is included in the old ^1.6.0 range, because of the caret ^ character .
When the test services report success Greenkeeper will silently delete the branch again, because no action needs to be taken – everything is fine.

However, should the tests fail, Greenkeeper will create an issue to inform you about the problem immediately.

This way, you’ll never be surprised by a dependency breaking your code. As long as everything still works, Greenkeeper will stay out of your way, and as soon as something goes wrong, you’ll be the first to know.

-  "lodash": "^3.0.0"
+  "lodash": "^4.0.0"

In this example, the new version 4.0.0 is not included in the old ^3.0.0 range.
For version updates like these – let’s call them “out of range” updates – you’ll receive a pull request.

This means that you no longer need to check for new versions manually – Greenkeeper will keep you up to date automatically.

These pull requests not only serve as reminders to update: If you have solid tests and good coverage, and the pull requests passes those tests, you can very likely just merge it and release a new version of your software straight away :shipit:

To get a better idea of which ranges apply to which releases, check out the extremely useful semver calculator provided by npm.

FAQ and help

There is a collection of frequently asked questions. If those don’t help, you can always ask the humans behind Greenkeeper.


Good luck with your project and see you soon ✨

Your Greenkeeper bot 🌴

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 1d99ae1 on greenkeeper/initial into 029b8dc on master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 1d99ae1 on greenkeeper/initial into 029b8dc on master.

Alt text is now "MIT License"

#1, #$
@gregswindle
Copy link
Owner Author

Modified estimate to account for greenkeeper's lockfile.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 5b390c9 on greenkeeper/initial into 029b8dc on master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 5b390c9 on greenkeeper/initial into 029b8dc on master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 5b390c9 on greenkeeper/initial into 029b8dc on master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 5b390c9 on greenkeeper/initial into 029b8dc on master.

@coveralls
Copy link

coveralls commented Nov 15, 2017

Coverage Status

Coverage remained the same at 100.0% when pulling 552b79f on greenkeeper/initial into eee3df0 on master.

Swindle, Gregory Jay (Greg) and others added 2 commits November 15, 2017 18:52
@coveralls
Copy link

coveralls commented Nov 16, 2017

Coverage Status

Coverage remained the same at 100.0% when pulling 6fb1ff2 on greenkeeper/initial into 40913bb on master.

@greenkeeper
Copy link

greenkeeper bot commented Nov 18, 2017

Hey there 👋,

we noticed that this PR isn’t merged yet. Just to let you know: if you don't merge this PR, Greenkeeper will not be enabled on this repo, and you won't receive updates for your dependencies.

If don't want to enable Greenkeeper here, just close or ignore this PR, we won't nag you again. ✌️

Have a great day! 🌴

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 7dd7d35 on greenkeeper/initial into 40913bb on master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 7dd7d35 on greenkeeper/initial into 40913bb on master.

@coveralls
Copy link

coveralls commented Nov 22, 2017

Coverage Status

Coverage remained the same at 100.0% when pulling 7dd7d35 on greenkeeper/initial into 40913bb on master.

gregswindle added a commit that referenced this pull request Dec 15, 2017
* chore(package): update dependencies

* docs(readme): add Greenkeeper badge

* docs(readme): add greenkeeper badge

Closes #1

* docs(license): edit license badge's alt text


Alt text is now "MIT License"

#1, #$

* docs(readme): init greenkeeper

Clean up README, too.

#1,#5

* ci(blackduck-copilot): enable CoPilot SaaS

* ci(dependency-ci): add config for removed libs

* fix(pom): Running "mvn site" results in error with message "Could not connect to Central search"

Applied the fix described here: jeremylong/DependencyCheck#978

#8

* build(mvnw): add /.mvn/ directory

#10

* docs(styleguide): add markdown styleguide

Format all Java source

#10,#11
@coveralls
Copy link

coveralls commented Dec 15, 2017

Coverage Status

Coverage remained the same at 100.0% when pulling 05dfea4 on greenkeeper/initial into 2a96fcd on master.

gregswindle added a commit that referenced this pull request Dec 17, 2017
* chore(license): might need to evaluate a LGLPL license.

* chore(package): update dependencies

* docs(readme): add Greenkeeper badge

* docs(readme): add greenkeeper badge

Closes #1

* docs(license): edit license badge's alt text


Alt text is now "MIT License"

#1, #$

* docs(readme): init greenkeeper

Clean up README, too.

#1,#5

* ci(blackduck-copilot): enable CoPilot SaaS

* ci(dependency-ci): add config for removed libs

* fix(pom): Running "mvn site" results in error with message "Could not connect to Central search"

Applied the fix described here: jeremylong/DependencyCheck#978

#8

* build(mvnw): add /.mvn/ directory

#10

* docs(styleguide): add markdown styleguide

Format all Java source

#10,#11

* docs(mvn-site): update with latest info

* ci(quality): add tooling

1. google-java-format: format all source code
2. javac-with-errorprone: static code analyzer
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants