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

Update CONTRIBUTING.md #1469

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

newarcher36
Copy link

Pull Request Description

This pull request closes #

Acceptance Test

  • Building the code with mvn clean install -Dintegration.tests still works.
  • Running mvn spring-boot:run in the strongbox-web-core still starts up the application correctly.
  • Building the code and running the strongbox-distribution from a zip or tar.gz still works.
  • The tests in the strongbox-web-integration-tests still run properly.

Questions

  • Does this pull request break backward compatibility?

    • Yes
    • No
  • Does this pull request require other pull requests to be merged first?

    • Yes, please see #...
    • No
  • Does this require an update of the documentation?

    • Yes, please see strongbox/strongbox-docs#{PR_NUMBER}
    • No

@ghost
Copy link

ghost commented Sep 21, 2019

DeepCode Report (#013586)

DeepCode analyzed this pull request.
There are no new issues.

@@ -144,3 +144,4 @@ To accept, please:
| Serdyuk Sergey | | Moscow, Russian Federation | 2019-06-20 |
| Bartosz Dabek | | Hajnowka, Poland | 2019-06-30 |
| Bogdan Sukonnov | | Saint Petersburg, Russia | 2019-08-19 |
| Daniel Jiménez | Open University of Catalonia | Barcelona, Catalonia, Spain | 2020-09-21 |
Copy link
Contributor

Choose a reason for hiding this comment

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

looks like you're coming from the future 😅

2020-09-21

Copy link
Member

Choose a reason for hiding this comment

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

Haha! Yeah! 😆

Copy link
Author

Choose a reason for hiding this comment

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

Hey guys! Sorry for my delayed response!
Ups, Hahaha! Thanks for fix this!

Copy link
Contributor

@BartoszDabek BartoszDabek left a comment

Choose a reason for hiding this comment

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

fix the contribution date

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.

None yet

3 participants