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

Amend the reference to JabRefReferences initialization #5487

Merged
merged 1 commit into from
Oct 20, 2019
Merged

Amend the reference to JabRefReferences initialization #5487

merged 1 commit into from
Oct 20, 2019

Conversation

cmoralesmx
Copy link
Contributor

The former filename does not exist, the updated file being referred does contain the initialization of JabRefPreferences.
If the previous name is kept, new contributors could potentially be confused when looking for the file.

I do not think this change qualifies as notable, therefore I did not include it in the CHANGELOG.md, it is just a correction to a comment. No tests were created either.


  • Change in CHANGELOG.md described
  • Tests created for changes
  • Manually tested changed features in running JabRef
  • Screenshots added in PR description (for bigger UI changes)
  • Ensured that the git commit message is a good one
  • Check documentation status (Issue created for outdated help page at help.jabref.org?)

The former filename does not exist, the updated file being referred does contain the initialization of JabRefPreferences.
If the previous name is kept, new contributors could potentially be confused when looking for the file.
@tobiasdiez
Copy link
Member

Thanks a lot for your contribution!

I guess you stumbled upon this while having a walk through the code. If you are looking for a starting point for your next walk, then https://github.com/JabRef/jabref/issues?q=is%3Aissue+is%3Aopen+label%3A%22good+first+issue%22 might be the right place for you.

@tobiasdiez tobiasdiez merged commit dfad2fd into JabRef:master Oct 20, 2019
@cmoralesmx cmoralesmx deleted the globals_JabRefPreferences branch October 20, 2019 17:52
Siedlerchr added a commit that referenced this pull request Oct 21, 2019
* upstream/master:
  Amend the reference to JabRefReferences initialization (#5487)
  Fix for issue 5463 (#5481)
  Rename index.md to README.md
Siedlerchr added a commit that referenced this pull request Oct 24, 2019
# By Tobias Diez (11) and others
# Via GitHub (30) and others
* upstream/master: (70 commits)
  Use Platform.runLater
  Fixes #5485
  Bump com.github.ben-manes.versions from 0.26.0 to 0.27.0
  Bump src/main/resources/csl-styles from `68a697b` to `c3fd4bd`
  Bump byte-buddy-parent from 1.10.1 to 1.10.2
  Bump mariadb-java-client from 2.5.0 to 2.5.1
  Bump classgraph from 4.8.48 to 4.8.52
  Bump org.beryx.jlink from 2.16.0 to 2.16.2
  Updated CHANGELOG.md
  Open entry editor by default on start-up
  Amend the reference to JabRefReferences initialization (#5487)
  Fix for issue 5463 (#5481)
  Rename index.md to README.md
  Fix not on fx thread error for custom entry types
  Refactor to sorted set (#5477)
  Removed a duplicate name (closes #5476)
  Remove unnecessary sort (#5470)
  lint CONTRIBUTING.md (#5473)
  Mark OOSTyle as invalid if no defaultStyle (#5471)
  Fix highlight problem in entry preview
  ...

# Conflicts:
#	build.gradle
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.

3 participants