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

Fix path of /docs/ [ci skip] #6075

Merged

Conversation

JustCris654
Copy link
Contributor

Summary

Fix the wrong mentions of the documentation folder which changed from /docs to /doc/user and in the function comments from /docs to /doc

@JustCris654 JustCris654 requested a review from a team as a code owner September 18, 2024 13:34
Copy link

codecov bot commented Sep 18, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 69.81%. Comparing base (b108104) to head (1bc92c3).
Report is 107 commits behind head on dev-2.x.

Additional details and impacted files
@@              Coverage Diff              @@
##             dev-2.x    #6075      +/-   ##
=============================================
+ Coverage      69.78%   69.81%   +0.02%     
- Complexity     17357    17418      +61     
=============================================
  Files           1962     1974      +12     
  Lines          74357    74537     +180     
  Branches        7623     7633      +10     
=============================================
+ Hits           51893    52039     +146     
- Misses         19820    19847      +27     
- Partials        2644     2651       +7     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@JustCris654
Copy link
Contributor Author

I added a commit to fix one last mention that I forgot before

@leonardehrenfried leonardehrenfried changed the title Fix path of /docs/ Fix path of /docs/ [ci skip] Sep 19, 2024
auto-merge was automatically disabled September 24, 2024 09:17

Head branch was pushed to by a user without write access

@JustCris654
Copy link
Contributor Author

Now the test should be correct and all the comments fixed

@habrahamsson-skanetrafiken
Copy link
Contributor

Great, thanks @JustCris654! I'll merge this since I guess you don't have merge rights

@habrahamsson-skanetrafiken habrahamsson-skanetrafiken merged commit 1c858c8 into opentripplanner:dev-2.x Sep 24, 2024
5 checks passed
@t2gran t2gran added this to the 2.7 (next release) milestone Oct 16, 2024
@JustCris654 JustCris654 deleted the fix-docs branch October 22, 2024 14:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants