Skip to content

Conversation

@rhrpr
Copy link
Contributor

@rhrpr rhrpr commented Sep 18, 2025

Description

Solves #39

  • Add rosetta compatibility layer command for use with FTP that is currently bundled as amd platform type only.
  • Correct readme step ordering that (step 9 referred to API key gen on step 6, which is now on step 7)
  • Wrap Token URLs in the Flexera One interface in markdown backticks and remove unicode characters, as when included these caused key creation error on FO key creation with that refresh token

@rhrpr rhrpr requested a review from a team as a code owner September 18, 2025 17:51
@rhrpr rhrpr added the FIX label Sep 18, 2025
@github-actions
Copy link

github-actions bot commented Sep 18, 2025

4 Warnings
⚠️

setup/linux/README.md

Outdated terminology found. Please remove references to defunct internal names for products or services:

Line 140: Reference to RightScale found. Recommended replacements: Flexera, Flexera CCO, Flexera Automation

⚠️

setup/macos/README.md

Outdated terminology found. Please remove references to defunct internal names for products or services:

Line 146: Reference to RightScale found. Recommended replacements: Flexera, Flexera CCO, Flexera Automation

⚠️

setup/windows_powershell/README.md

Outdated terminology found. Please remove references to defunct internal names for products or services:

Line 147: Reference to RightScale found. Recommended replacements: Flexera, Flexera CCO, Flexera Automation

⚠️

setup/windows_wsl2/README.md

Outdated terminology found. Please remove references to defunct internal names for products or services:

Line 140: Reference to RightScale found. Recommended replacements: Flexera, Flexera CCO, Flexera Automation

1 Message
📖

Github Pull Request

Pull Request is missing READY-FOR-REVIEW label. Please add this label if this Pull Request is ready for review.

Please note that this message may be a false positive if you've added the label after Dangerfile tests were run, since adding labels does not trigger them to run again. In these cases, simply ignore this message.

Generated by 🚫 Danger

Copy link

@bryankaraffa bryankaraffa left a comment

Choose a reason for hiding this comment

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

Thanks for the edits. Looks good to me

@bryankaraffa bryankaraffa merged commit 01a2276 into main Sep 25, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants