Skip to content

Cleanup EMACS code#2317

Merged
lenhard merged 1 commit into
masterfrom
emacsclient
Nov 29, 2016
Merged

Cleanup EMACS code#2317
lenhard merged 1 commit into
masterfrom
emacsclient

Conversation

@koppor

@koppor koppor commented Nov 28, 2016

Copy link
Copy Markdown
Member
  • This PR removes support of EMACS<23 as this version of emacs disappeared even in debian/stable.

  • The default value for emacs path is emacsclient.

  • Default preferences: Also reorders default values to be consistent on each branch

  • [no] Change in CHANGELOG.md described

  • [no] Tests created for changes

  • [no] Screenshots added (for bigger UI changes)

  • Manually tested changed features in running JabRef

  • Check documentation status (Issue created for outdated help page at help.jabref.org?)

  • If you changed the localization: Did you run gradle localizationUpdate?

@koppor koppor added the status: ready-for-review Pull Requests that are ready to be reviewed by the maintainers label Nov 28, 2016
 - Remove support of EMACS<23
 - Use emacsclient on linux, too
 - Default preferences: Also reorders default values to be consistent on each branch
@lenhard

lenhard commented Nov 29, 2016

Copy link
Copy Markdown
Member

Code-wise, this looks fine. Lacking emacs, I cannot check this locally, but I'll trust you on it.

@lenhard lenhard merged commit 94ac74a into master Nov 29, 2016
@lenhard lenhard deleted the emacsclient branch November 29, 2016 16:58
Siedlerchr added a commit that referenced this pull request Nov 29, 2016
* upstream/master:
  Cleanup EMACS code (#2317)
  Update mockito-core from 2.2.15 to 2.2.21
  Fix typo in comment
Siedlerchr added a commit that referenced this pull request Dec 2, 2016
* upstream/master:
  Ignore failing test
  Replace usage of Threads and priorities with thread pool (#2304)
  Class variable declarations and method declarations are now separated by one line
  Disable joining of wrapped lines
  Installer Code Signing #1879 (#2320)
  Add bibtex key deviation check (#2328)
  Update mockito-core (2.2.21 -> 2.2.26) and wiremock (2.3.1 -> 2.4.1)
  Fix opening of preference dialog with Java 9 (#2329)
  Add longer explanation for ID-based entry generation. (#2330)
  Add DOI integrity check (#2327)
  New strings translated (#2325)
  Fix exporting via commandline in no gui mode (#2316)
  Cleanup EMACS code (#2317)
  Update mockito-core from 2.2.15 to 2.2.21
  Fix typo in comment
  Updated JabRef_tr.properties (#2315)

# Conflicts:
#	CHANGELOG.md
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status: ready-for-review Pull Requests that are ready to be reviewed by the maintainers

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants