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

Dash in list of illegal chars #6299

Closed
wants to merge 3 commits into from

Conversation

dextep
Copy link
Contributor

@dextep dextep commented Apr 16, 2020

  • Change in CHANGELOG.md described (if applicable)
  • Tests created for changes (if applicable)
  • Manually tested changed features in running JabRef (always required)
  • Screenshots added in PR description (for UI changes)
  • Checked documentation: Is the information available and up to date? If not created an issue at https://github.com/JabRef/user-documentation/issues or, even better, submitted a pull request to the documentation repository.

@dextep dextep closed this Apr 16, 2020
@dextep dextep reopened this Apr 16, 2020
@dextep dextep closed this Apr 16, 2020
@dextep dextep deleted the dash-in-list-of-illegal-chars branch April 16, 2020 19:43
@Siedlerchr
Copy link
Member

First update your fork by merging in upstream/master
Push to origin
Then create a new branch and do the fix
Then push the branch
Then all should be fine

@dextep
Copy link
Contributor Author

dextep commented Apr 16, 2020

Thanks @Siedlerchr! That was a bit confused for me, sorry for that. Hope right now is fine :)

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.

2 participants