Skip to content

Add instructions on how to use the automatic GHA token#263

Merged
ramsey merged 5 commits intoramsey:v3from
Jean85:patch-2
Mar 5, 2025
Merged

Add instructions on how to use the automatic GHA token#263
ramsey merged 5 commits intoramsey:v3from
Jean85:patch-2

Conversation

@Jean85
Copy link
Copy Markdown
Contributor

@Jean85 Jean85 commented Feb 7, 2025

As demonstrated in facile-it/paraunit@3e253cf, we can leverage the GHA automatic token to download public forks.

Description

Added instructions in the readme on how to download public forks without creating a PAT.

How has this been tested?

See commit above as test.

PR checklist

  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING.md document.
  • I have added tests to cover my changes.

As demonstrated in facile-it/paraunit@3e253cf, we can leverage the GHA automatic token to download public forks.
@Jean85 Jean85 requested a review from ramsey as a code owner February 7, 2025 17:08
@codecov
Copy link
Copy Markdown

codecov bot commented Mar 5, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 95.58%. Comparing base (2da6381) to head (25bbc41).
Report is 1 commits behind head on v3.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##               v3     #263      +/-   ##
==========================================
+ Coverage   93.38%   95.58%   +2.20%     
==========================================
  Files           6        6              
  Lines         136      136              
==========================================
+ Hits          127      130       +3     
+ Misses          9        6       -3     

see 2 files with indirect coverage changes

🚀 New features to boost your workflow:
  • Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Copy Markdown
Owner

@ramsey ramsey left a comment

Choose a reason for hiding this comment

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

Thanks, @Jean85! I made some light editorial changes; I hope you don't mind. 🙂

@ramsey ramsey merged commit a2636af into ramsey:v3 Mar 5, 2025
32 checks passed
@Jean85 Jean85 deleted the patch-2 branch March 6, 2025 08:46
@Jean85
Copy link
Copy Markdown
Contributor Author

Jean85 commented Mar 6, 2025

Thanks, @Jean85! I made some light editorial changes; I hope you don't mind. 🙂

Thank you, that's ok! 😄

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