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.

Sign up for GitHub

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

Jump to bottom

Update actions node vesion and update templates #475

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

Update actions node vesion and update templates #475

wants to merge 2 commits into from

Conversation

dobromirts
Copy link
Contributor

@dobromirts dobromirts commented Feb 21, 2025 •
edited
Loading

Closes #476
Closes #477
Closes #478

  1. Fixing node security vulnerability based on child_process.spawn and child_process.spawnSync - https://nodejs.org/en/blog/vulnerability/april-2024-security-releases-2
  2. Fixed multiple Table of content request made for AB documentation
  3. Updated tml partial files with Copyright year
  4. Added automatical update copyright year in footer templates
  • Added _currentYear to model in conceptual.html.primary.js
  • Used {{_currentYear}} in the footer template to dynamically display the current year
  • Ensures the copyright year updates automatically with each DocFX build

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@zdrawku zdrawku Awaiting requested review from zdrawku

@hanastasov hanastasov Awaiting requested review from hanastasov

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
status: awaiting-test
Projects
None yet
Milestone
No milestone
1 participant
@dobromirts