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

[deps] Platform (CL): Update @compodoc/compodoc to v1.1.25 #8966

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

renovate[bot]
Copy link
Contributor

@renovate renovate bot commented Apr 29, 2024

Mend Renovate

This PR contains the following updates:

Package Change Age Adoption Passing Confidence
@compodoc/compodoc 1.1.23 -> 1.1.25 age adoption passing confidence

Release Notes

compodoc/compodoc (@​compodoc/compodoc)

v1.1.25

Compare Source

Merged
  • style(templates): Fix typo in dark mode comments #​1476, Thanks Keiishu
Bug fixes

v1.1.24

Compare Source

Merged
Bug fixes

Configuration

📅 Schedule: Branch creation - "every 2nd week starting on the 2 week of the year before 4am on Monday" (UTC), Automerge - At any time (no schedule defined).

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 Ignore: Close this PR and you won't be reminded about this update again.


  • If you want to rebase/retry this PR, check this box

This PR has been generated by Mend Renovate. View repository job log here.

@renovate renovate bot requested a review from a team April 29, 2024 03:03
Copy link

codecov bot commented Apr 29, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 28.15%. Comparing base (868e0a5) to head (30cf56a).
Report is 41 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #8966      +/-   ##
==========================================
- Coverage   28.18%   28.15%   -0.03%     
==========================================
  Files        2397     2422      +25     
  Lines       70642    71348     +706     
  Branches    13203    13319     +116     
==========================================
+ Hits        19910    20091     +181     
- Misses      49172    49679     +507     
- Partials     1560     1578      +18     

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

@renovate renovate bot force-pushed the renovate/compodoc-compodoc-1.x branch from 8fa3bd2 to b82f474 Compare May 29, 2024 15:53
@renovate renovate bot changed the title [deps] Platform (CL): Update @compodoc/compodoc to v1.1.24 [deps] Platform (CL): Update @compodoc/compodoc to v1.1.25 May 29, 2024
@renovate renovate bot force-pushed the renovate/compodoc-compodoc-1.x branch from b82f474 to fcc662f Compare May 30, 2024 22:44
@vleague2 vleague2 requested review from a team as code owners June 4, 2024 18:32
@vleague2 vleague2 requested a review from r-tome June 4, 2024 18:32
Copy link
Contributor Author

renovate bot commented Jun 4, 2024

Edited/Blocked Notification

Renovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR.

You can manually request rebase by checking the rebase/retry box above.

⚠️ Warning: custom changes will be lost.

@vleague2 vleague2 requested a review from willmartian June 4, 2024 18:38
@vleague2
Copy link
Contributor

vleague2 commented Jun 4, 2024

ℹ️ While testing the version update, I noticed that the npm run docs:json command was erroring. First there were some command line flags missing, and secondly there were four files that did not have template references and appear to be unused. Removing those allows compodoc to run successfully. The web app looks to build and run fine locally -- but please let me know if we do need these files.

Copy link
Contributor

@willmartian willmartian left a comment

Choose a reason for hiding this comment

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

LGTM, ty Vicki

This comment was marked as off-topic.

@eliykat eliykat requested review from eliykat and removed request for r-tome June 13, 2024 05:05
Copy link
Member

@eliykat eliykat left a comment

Choose a reason for hiding this comment

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

I've come across similar unused files before - iirc, they're leftover navigation components from pre-vertical nav refresh. LGTM.

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.

None yet

3 participants