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

Move server shortcodes to chef-server #3295

Merged
merged 5 commits into from May 12, 2022
Merged

Move server shortcodes to chef-server #3295

merged 5 commits into from May 12, 2022

Conversation

IanMadd
Copy link
Contributor

@IanMadd IanMadd commented May 5, 2022

Description

  • Move Server shortcodes from chef-web-docs to chef-server repo.
  • Use the readfile shortcode to add files to docs instead of the readFile_shortcode shortcode where necessary.
  • Update the readme to describe where to put shortcodes
  • Delete shortcodes used only once and place the text from the shortcode in the doc.

Issues Resolved

https://chefio.atlassian.net/browse/CHEFDOCS-179

Check List

@IanMadd IanMadd added the Documentation Pulls PR onto docs board so they know it exists label May 5, 2022
@IanMadd IanMadd requested a review from a team as a code owner May 5, 2022 20:13
@netlify
Copy link

netlify bot commented May 5, 2022 •
edited

👷 Deploy Preview for chef-server processing.

Name Link
🔨 Latest commit f92f59a
🔍 Latest deploy log https://app.netlify.com/sites/chef-server/deploys/627d2137691bc20008d9306d

@IanMadd IanMadd changed the title Move server shortcodes to chef-server Move server shortcodes to chef-server WIP May 5, 2022
@IanMadd IanMadd changed the title Move server shortcodes to chef-server WIP Move server shortcodes to chef-server May 6, 2022
@IanMadd @marcparadise
Update shortcodes readme
4ebf334 
Signed-off-by: Ian Maddaus <ian.maddaus@progress.com>
@IanMadd @marcparadise
Move server shortcodes to chef-server
4698aa3 
Signed-off-by: Ian Maddaus <ian.maddaus@progress.com>
@IanMadd @marcparadise
Improve Netlify preview make script
dfe22e6 
Signed-off-by: Ian Maddaus <ian.maddaus@progress.com>
@IanMadd @marcparadise
Update hugo version
4700a79 
Signed-off-by: Ian Maddaus <ian.maddaus@progress.com>
@IanMadd @marcparadise
Remove Hugo version from netlify config file
f92f59a 
Signed-off-by: Ian Maddaus <ian.maddaus@progress.com>
@sonarcloud
Copy link

sonarcloud bot commented May 12, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@marcparadise marcparadise merged commit 9019c1e into main May 12, 2022
@marcparadise marcparadise deleted the im/reorganize_shortcodes branch May 12, 2022 19:12
IanMadd added a commit that referenced this pull request May 12, 2022
@IanMadd
More cleanup related to #3295
85c135c 
Signed-off-by: Ian Maddaus <ian.maddaus@progress.com>
@IanMadd IanMadd mentioned this pull request May 12, 2022
Merged
5 tasks
marcparadise added a commit that referenced this pull request May 13, 2022
@marcparadise
Merge pull request #3312 from chef/im/additional_docs_cleanup
2c47843 
More cleanup related to #3295
This was referenced May 18, 2022
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@marcparadise marcparadise marcparadise approved these changes

Assignees
No one assigned
Labels
Documentation Pulls PR onto docs board so they know it exists
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@IanMadd @marcparadise