-
Notifications
You must be signed in to change notification settings - Fork 10
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
Provides multiple language previews #26
base: main
Are you sure you want to change the base?
Conversation
repository: "readthedocs/actions" | ||
ref: "v1" | ||
repository: "Yaminyam/actions" | ||
ref: "main" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This part has been modified for test and must be returned before merging.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See #28
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey! Thanks for sending these changes! I think they are really good. I left some suggestions to be considered before moving forward with it 👍🏼
preview/scripts/edit-description.js
Outdated
@@ -1,7 +1,7 @@ | |||
module.exports = async ({inputs, github, context}) => { | |||
const PR_NUMBER = context.issue.number; | |||
const RTD_PROJECT_SLUG = inputs["project-slug"]; | |||
const RTD_PROJECT_LANGUAGE = inputs["project-language"]; | |||
const RTD_PROJECT_LANGUAGES = inputs["project-languages"].split(","); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Instead of splitting it by ,
, this attribute could be a list, right? Like
project-languages:
- en
- ja
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@humitos Please review! |
I demonstrated it as follows.
lablup/backend.ai#1139
Please feel free to let me know if there is anything to improve!
close #25
📚 Documentation preview 📚: https://readthedocs-preview--26.org.readthedocs.build/en/26/