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

Internal: Update HydeStan workflow to check documentation formatting #1452

Closed
wants to merge 6 commits into from

Conversation

caendesilva
Copy link
Member

@caendesilva caendesilva commented Nov 12, 2023

Updates the HydeStan workflow to check documentation formatting. Also refactors the formatter to work without Composer dependencies to speed things up.

Copy link

codecov bot commented Nov 12, 2023

Codecov Report

Merging #1452 (6eaa92b) into master (0594ddc) will not change coverage.
Report is 11 commits behind head on master.
The diff coverage is n/a.

@@              Coverage Diff              @@
##              master     #1452     +/-   ##
=============================================
  Coverage     100.00%   100.00%             
- Complexity      1728      3456   +1728     
=============================================
  Files            180       360    +180     
  Lines           4693      9386   +4693     
=============================================
+ Hits            4693      9386   +4693     

see 181 files with indirect coverage changes

@caendesilva caendesilva force-pushed the documentation-static-analysis branch from 9bd6ecd to dfc39b2 Compare November 12, 2023 15:22
@caendesilva caendesilva temporarily deployed to pr-documentation-1452 November 12, 2023 15:22 — with GitHub Actions Inactive
@caendesilva caendesilva temporarily deployed to pr-documentation-1452 November 12, 2023 15:22 — with GitHub Actions Inactive
@caendesilva
Copy link
Member Author

No idea why the CI is acting up. Not worth working on ATM.

@caendesilva caendesilva deleted the documentation-static-analysis branch November 12, 2023 16:28
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