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

chore(ia): reorg /web/guide dir #37790

Merged
merged 5 commits into from
Jan 29, 2025
Merged

chore(ia): reorg /web/guide dir #37790

merged 5 commits into from
Jan 29, 2025
+24 −18

Conversation

bsmth
Copy link
Member

@bsmth bsmth commented Jan 24, 2025 •
edited
Loading

Description

In preparation for getting rid of the /web/guide dir, these resources need a new home.
There is no real content change, this is moving docs only, with one addition for sidebars.

Motivation

IA improvements & maintenance
Eventually, the guide dir will be gone, but this is blocked on #37784 so I'm taking care of the other pages first.

TODO:

  • revert reordering of sidebars from fmt-sidebars command

@bsmth bsmth requested review from a team as code owners January 24, 2025 16:53
@bsmth bsmth requested review from chrisdavidmills and removed request for a team January 24, 2025 16:53
@github-actions github-actions bot added Content:CSS Cascading Style Sheets docs Content:SVG SVG docs Content:Guide Guide docs size/s [PR only] 6-50 LoC changed labels Jan 24, 2025
bsmth
bsmth commented Jan 24, 2025
View reviewed changes
files/sidebars/cssref.yaml Outdated Show resolved Hide resolved
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Jan 24, 2025 •
edited
Loading

Preview URLs

(comment last updated: 2025-01-28 15:52:45)

@bsmth bsmth requested a review from pepelsbey January 27, 2025 16:00
@sideshowbarker sideshowbarker removed the request for review from a team January 28, 2025 03:32
@github-actions github-actions bot added the Content:Learn Learning area docs label Jan 28, 2025
@bsmth bsmth requested a review from caugner January 28, 2025 16:12
pepelsbey
pepelsbey approved these changes Jan 28, 2025
View reviewed changes
Copy link
Member

@pepelsbey pepelsbey left a comment

Choose a reason for hiding this comment

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

Looks good! Thank you :)

@chrisdavidmills chrisdavidmills removed their request for review January 28, 2025 16:35
@bsmth
Copy link
Member Author

bsmth commented Jan 29, 2025

Thank you!

@bsmth bsmth merged commit de6055c into mdn:main Jan 29, 2025
9 checks passed
@bsmth bsmth deleted the move-content-guide branch January 29, 2025 08:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pepelsbey pepelsbey pepelsbey approved these changes

@caugner caugner Awaiting requested review from caugner

Assignees
No one assigned
Labels
Content:CSS Cascading Style Sheets docs Content:Guide Guide docs Content:Learn Learning area docs Content:SVG SVG docs size/s [PR only] 6-50 LoC changed
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@bsmth @pepelsbey @caugner