Skip to content

Update README.md#590

Merged
KrzysztofPajak merged 1 commit intograndnode:mainfrom
GuyPago:patch-1
Mar 5, 2025
Merged

Update README.md#590
KrzysztofPajak merged 1 commit intograndnode:mainfrom
GuyPago:patch-1

Conversation

@GuyPago
Copy link
Copy Markdown
Contributor

@GuyPago GuyPago commented Mar 4, 2025

Resolves #issueNumber
Type: feature|bugfix|

Issue

  1. Typo.
  2. License is linked on http
  3. Some code blocks miss syntax highlighting. For example:
[Unit]
Description=GrandNode

[Service]
WorkingDirectory=/var/webapps/grandnode
ExecStart=/usr/bin/dotnet /var/webapps/grandnode/Grand.Web.dll
Restart=always
RestartSec=10
SyslogIdentifier=dotnet-grandnode
User=www-data
Environment=ASPNETCORE_ENVIRONMENT=Production

[Install]
WantedBy=multi-user.target

Solution

  1. FIx typo.
  2. Use local license link.
  3. Add some syntax highlighting. For example:
[Unit]
Description=GrandNode

[Service]
WorkingDirectory=/var/webapps/grandnode
ExecStart=/usr/bin/dotnet /var/webapps/grandnode/Grand.Web.dll
Restart=always
RestartSec=10
SyslogIdentifier=dotnet-grandnode
User=www-data
Environment=ASPNETCORE_ENVIRONMENT=Production

[Install]
WantedBy=multi-user.target

Breaking changes

Testing

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Mar 4, 2025

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@KrzysztofPajak
Copy link
Copy Markdown
Member

KrzysztofPajak commented Mar 5, 2025

@GuyPago Thanks!

@KrzysztofPajak KrzysztofPajak merged commit 7466434 into grandnode:main Mar 5, 2025
5 checks passed
szjanikowski pushed a commit to NoesisVision/grandnode2 that referenced this pull request May 8, 2025
@GuyPago
Update README.md (grandnode#590)
be72592
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@GuyPago @KrzysztofPajak