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 Install-LibreNMS.md #12201

Merged
merged 1 commit into from
Oct 8, 2020
Merged

Update Install-LibreNMS.md #12201

merged 1 commit into from
Oct 8, 2020

Conversation

mjeffin
Copy link
Contributor

@mjeffin mjeffin commented Oct 8, 2020

Change the symbol for comments from # to ;
This threw an error for me while following the steps and had to google to fix the issue. Changing to ; might help people like me who are new to php. Not sure if it would affect any older versions.

Please give a short description what your pull request is for

DO NOT DELETE THE UNDERLYING TEXT

Please note

Please read this information carefully. You can run ./lnms dev:check to check your code before submitting.

  • Have you followed our code guidelines?
  • If my Pull Request does some changes/fixes/enhancements in the WebUI, I have inserted a screenshot of it.

Testers

If you would like to test this pull request then please run: ./scripts/github-apply <pr_id>, i.e ./scripts/github-apply 5926
After you are done testing, you can remove the changes with ./scripts/github-remove. If there are schema changes, you can ask on discord how to revert.

Change the symbol for comments from # to ; 
This threw an error for me while following the steps and had to google to fix the issue. Changing to ; might help people like me who are new to php. Not sure if it would affect any older versions.
@CLAassistant
Copy link

CLAassistant commented Oct 8, 2020

CLA assistant check
All committers have signed the CLA.

Copy link
Member

@SourceDoctor SourceDoctor left a comment

Choose a reason for hiding this comment

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

good point
thanks

@SourceDoctor SourceDoctor merged commit 90483b8 into librenms:master Oct 8, 2020
@murrant
Copy link
Member

murrant commented Oct 9, 2020

Interesting, I didn't consider that people would copy those lines :D

@mjeffin
Copy link
Contributor Author

mjeffin commented Oct 9, 2020 via email

@murrant
Copy link
Member

murrant commented Oct 9, 2020

Anything that makes it easier is appreciated, thanks.

@murrant
Copy link
Member

murrant commented Nov 2, 2020

This pull request has been mentioned on LibreNMS Community. There might be relevant details there:

https://community.librenms.org/t/v1-69-release-changelog-october-2020/13837/1

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.

4 participants