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

Fixed typos in cassandra.ini #2495

Closed
wants to merge 3 commits into from
Closed

Conversation

arampaa
Copy link
Contributor

@arampaa arampaa commented Mar 13, 2019

Basics

Do not describe the purpose here but:

  • Short descriptions should be in the release notes (added as entry in
    doc/news/_preparation_next_release.md which contains _(my name)_)
    Please always add something to the the release notes.
  • Longer descriptions should be in documentation or in design decisions.
  • Describe details of how you changed the code in commit messages
    (first line should have module: short statement syntax)
  • References to issues, e.g. close #X, should be in the commit messages.

Checklist

Check relevant points but please do not remove entries.
For docu fixes, spell checking, and similar none of these points below
need to be checked.

  • I added unit tests
  • I ran all tests locally and everything went fine
  • affected documentation is fixed
  • I added code comments, logging, and assertions (see doc/CODING.md)
  • meta data is updated (e.g. README.md of plugins)

Review

Remove the line below and add the "work in progress" label if you do
not want the PR to be reviewed:

"work in progress"

@markus2330
Copy link
Contributor

Thank you for creating this PR! I added you to the private repo.

Please add yourself into the release notes as instructed by the build server (Syntax is wrong).

@markus2330
Copy link
Contributor

@Piankero can you fix this PR please?

sanssecours added a commit to sanssecours/elektra that referenced this pull request Jul 3, 2019
sanssecours added a commit that referenced this pull request Jul 4, 2019
Fixed Typos in `cassandra.ini` (Updated Version of PR #2495)
@sanssecours
Copy link
Member

@arampaa Thank you for your contribution 💕. I added a slightly modified version of your changes to the master branch in PR #2818, since the Jenkins build job for this PR did not finish successfully, even after I restarted it multiple times.

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

3 participants