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

Minor updates to the porting readme #881

Merged
merged 5 commits into from
Sep 16, 2024
Merged

Conversation

rogurr
Copy link
Contributor

@rogurr rogurr commented Sep 13, 2024

Ran through the porting steps on a clean WSL and found a few minor corrections to the documentation.

@Javagedes
Copy link
Contributor

Hi @rogurr, this is great; thank you! There is currently a markdown issue outside of your change. I'm going to get that fixed and then get your PR merged.

Copy link

codecov bot commented Sep 16, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 78.71%. Comparing base (bf643b0) to head (2e11c3f).
Report is 102 commits behind head on master.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #881      +/-   ##
==========================================
- Coverage   78.75%   78.71%   -0.05%     
==========================================
  Files          49       41       -8     
  Lines        4909     5050     +141     
==========================================
+ Hits         3866     3975     +109     
- Misses       1043     1075      +32     

see 49 files with indirect coverage changes

@Javagedes Javagedes added the documentation Improvements or additions to documentation label Sep 16, 2024
@Javagedes Javagedes added this to the v0.27.13 milestone Sep 16, 2024
@Javagedes Javagedes merged commit 89c6416 into tianocore:master Sep 16, 2024
24 of 25 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants