Skip to content

Commit

Permalink
misc: prefix ValidationTask with I, it's an interface
Browse files Browse the repository at this point in the history
Mention in PR comment script that you now need to be logged into GitHub to download artifacts.
  • Loading branch information
GreemDev committed Dec 31, 2024
1 parent a5cde8e commit 318498e
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/nightly_pr_comment.yml
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@ jobs:
if (!artifacts.length) {
return core.error(`No artifacts found`);
}
let body = `Download the artifacts for this pull request:\n`;
let body = `*You need to be logged into GitHub to download these files.*\n\nDownload the artifacts for this pull request:\n`;
let hidden_debug_artifacts = `\n\n <details><summary>Only for Developers</summary>\n`;
for (const art of artifacts) {
const url = `https://github.com/Ryubing/Ryujinx/actions/runs/${run_id}/artifacts/${art.id}`;
Expand Down
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
namespace Ryujinx.BuildValidationTasks
{
public interface ValidationTask
public interface IValidationTask
{
public bool Execute(string projectPath, bool isGitRunner);
}
Expand Down
2 changes: 1 addition & 1 deletion src/Ryujinx.BuildValidationTasks/LocalesValidationTask.cs
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@

namespace Ryujinx.BuildValidationTasks
{
public class LocalesValidationTask : ValidationTask
public class LocalesValidationTask : IValidationTask
{
public LocalesValidationTask() { }

Expand Down

0 comments on commit 318498e

Please sign in to comment.