Skip to content

update help message#776

Merged
Axel-Naumann merged 3 commits intomainfrom
ferdymercury-patch-1
Jul 16, 2022
Merged

update help message#776
Axel-Naumann merged 3 commits intomainfrom
ferdymercury-patch-1

Conversation

@ferdymercury
Copy link
Contributor

@ferdymercury ferdymercury commented Apr 7, 2022

@ferdymercury ferdymercury marked this pull request as ready for review April 7, 2022 16:14
@ferdymercury ferdymercury requested a review from couet April 7, 2022 16:14
@ferdymercury
Copy link
Contributor Author

It complements these other PRs whose merge is pending:
#775
#773

@ferdymercury ferdymercury added the documentation Improvements or additions to documentation label Apr 13, 2022
@ferdymercury
Copy link
Contributor Author

@couet I think the failing check is unrelated to the PR ?

@couet
Copy link
Member

couet commented Apr 21, 2022

It is weird because the two links mentioned by the HTML proofer seems correct:
https://github.com/root-project/web/runs/6106813603?check_suite_focus=true
I ran the build again and still the same error shows.
May be @eguiraud will have an idea.

@couet couet requested a review from eguiraud April 21, 2022 05:48
@eguiraud
Copy link
Contributor

@ferdymercury please rebase on the latest main, should be solved

@ferdymercury ferdymercury force-pushed the ferdymercury-patch-1 branch from 9e9c587 to cc3fb5b Compare April 21, 2022 14:17
@ferdymercury
Copy link
Contributor Author

@ferdymercury please rebase on the latest main, should be solved

done, thanks!

Copy link
Member

@couet couet left a comment

Choose a reason for hiding this comment

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

LGTM

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.

4 participants