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: Update gem setting to latest slate version #2257

Merged
merged 9 commits into from
Apr 27, 2023
Merged

Conversation

strehle
Copy link
Member

@strehle strehle commented Mar 23, 2023

Updated Gem* to latest vesion from https://github.com/slatedocs/slate , e.g.
https://github.com/slatedocs/slate/blob/main/Gemfile
https://github.com/slatedocs/slate/blob/main/Gemfile.lock

Check of result can be seen with:

docker run --tty --interactive --workdir /root/uaa --volume $PWD:/root/uaa --env DB=hsqldb --rm cfidentity/uaa-generate-docs /root/uaa/scripts/generate-docs.sh

Compare result from ./uaa/uaa/build/docs/....
with
https://docs.cloudfoundry.org/api/uaa/

Currently I see some formatting changes but content is same and it would solve
https://github.com/cloudfoundry/uaa/security/dependabot

@cf-gitbot
Copy link

We have created an issue in Pivotal Tracker to manage this:

https://www.pivotaltracker.com/story/show/184780985

The labels on this github issue will be updated when the story is started.

@strehle strehle added ruby Pull requests that update Ruby code dependencies Pull requests that update a dependency file documentation labels Mar 24, 2023
@strehle strehle changed the title fix: Update gem setting to latest slate version [WIP,fix]: Update gem setting to latest slate version Mar 24, 2023
@Tallicia
Copy link
Contributor

@bruce-ricard If the build succeeded locally for you, are we comfortable merging this?

@strehle
Copy link
Member Author

strehle commented Mar 29, 2023

@bruce-ricard If the build succeeded locally for you, are we comfortable merging this?

please do not merge this one .... we need to sync more from slate, e.g.
slatedocs/slate#1226

@Tallicia Tallicia added blocked Issue or PR is blocked and removed unscheduled in progress labels Apr 14, 2023
@strehle strehle requested a review from a team April 19, 2023 15:45
@strehle strehle changed the title [WIP,fix]: Update gem setting to latest slate version fix: Update gem setting to latest slate version Apr 19, 2023
@strehle strehle added the in_review The PR is currently in review label Apr 19, 2023
@strehle
Copy link
Member Author

strehle commented Apr 19, 2023

@Tallicia locally I now see same formatting/content compared to https://docs.cloudfoundry.org/api/uaa, so from my side, please check and then we could finish with this topic
Added some files from https://github.com/slatedocs/slate/pull/1226/files

Copy link
Contributor

@Tallicia Tallicia left a comment

Choose a reason for hiding this comment

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

Formatting additions look good.

@strehle strehle added this to the 76.11.0 milestone Apr 27, 2023
@strehle strehle merged commit 491b463 into develop Apr 27, 2023
@strehle strehle deleted the slateUpdate branch April 27, 2023 10:09
@strehle strehle linked an issue Apr 27, 2023 that may be closed by this pull request
@Tallicia Tallicia restored the slateUpdate branch April 27, 2023 18:11
strehle added a commit that referenced this pull request Apr 27, 2023
* fix: Update gem setting to latest slate version

* gh checker

* revert some bundle versions

* locally working now with docker

* add bundler 2 dependency to gradle

* Update from rouge

https://github.com/slatedocs/slate/pull/1226/files
Tallicia pushed a commit that referenced this pull request Apr 28, 2023
* fix: Update gem setting to latest slate version

* gh checker

* revert some bundle versions

* locally working now with docker

* add bundler 2 dependency to gradle

* Update from rouge

https://github.com/slatedocs/slate/pull/1226/files
@cf-gitbot cf-gitbot added delivered accepted Accepted the issue and removed delivered labels Jun 27, 2023
@strehle strehle deleted the slateUpdate branch December 1, 2023 12:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
accepted Accepted the issue blocked Issue or PR is blocked dependencies Pull requests that update a dependency file documentation in_review The PR is currently in review ruby Pull requests that update Ruby code
Projects
Development

Successfully merging this pull request may close these issues.

Security issues with v76.6.0 (2 Critical, 7 High, 2 Medium)
4 participants