Skip to content

Update installing-license-to-ci-services.md #3036

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

Closed
wants to merge 1 commit into from

Conversation

MarcWeintraub
Copy link

No description provided.

@LanceMcCarthy
Copy link
Contributor

Hi Marc, thank you for taking the time to offer a suggestion. This section needs to be indented due to the rendering system, however there is a way to resolve this

The double brackets need to be escaped in quotes, like this:

# broken
TELERIK_LICENSE: ${{ secrets.TELERIK_LICENSE }}

#fixed
TELERIK_LICENSE: ${{ "{{ secrets.TELERIK_LICENSE }}" }}

I had already opened another one before I saw yours, so you don't need to take any action. Our WPF team will take care of things from here (cc @martinivanoff this PR can be closed after merging #3037).

Caution

This is only needed for our documentation source, your GitHub Workflow should be using the first option.

@MarcWeintraub
Copy link
Author

Thanks for the reply. I'd recommend updating other documentations also because I took this formatting from another document.

@LanceMcCarthy
Copy link
Contributor

LanceMcCarthy commented Apr 2, 2025

I'd recommend updating other documentations also because I took this formatting from another document.

I immediately reached out to all the product teams at the moment I realized what the exact issue was in our support case :) Again, thank you so much for your willingness to assist and being proactive!

@martinivanoff
For reference, see this post (note: that is a Microsoft Teams link that will only work for employees in the DevTools group).

@martinivanoff
Copy link
Contributor

Thank you for the report guys. Already merged Lance's pull request and will re-upload the docs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants