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

Platform as a service #3127

Closed
wants to merge 2 commits into from
Closed

Conversation

manojrayar
Copy link

Describe your changes

Added the new term Platform as a service.

Related issue number or link (ex: resolves #issue-number)

#3036

Checklist before opening this PR (put x in the checkboxes)

  • [x ] This PR does not contain plagiarism
    • don’t copy other people’s work unless you are quoting and contributing it to them.
  • [x ] I have signed off on all commits
    • signing off (ex: git commit -s) is to affirm that commits comply DCO. If you are working locally, you could add an alias to your gitconfig by running git config --global alias.ci "commit -s".

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
Signed-off-by: Mallikarjun Rayar <[email protected]>

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
Signed-off-by: Mallikarjun Rayar <[email protected]>
Copy link

netlify bot commented May 8, 2024

Deploy Preview for cncfglossary ready!

Name Link
🔨 Latest commit e7da058
🔍 Latest deploy log https://app.netlify.com/sites/cncfglossary/deploys/663b36baacb3150009df7620
😎 Deploy Preview https://deploy-preview-3127--cncfglossary.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link
Collaborator

@iamNoah1 iamNoah1 left a comment

Choose a reason for hiding this comment

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

@manojrayar thanks, ptal at the comment I made in the issue, thanks

Copy link
Collaborator

@Okabe-Junya Okabe-Junya left a comment

Choose a reason for hiding this comment

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

see #3218 (review)

I am not an official reviewer for the English version, but...

It looks like you have two PRs open for the PaaS page - the other is #3127. One page is sufficient. It would be better to close one of them, thanks

@jihoon-seo
Copy link
Collaborator

This PR is to add a new file content/en/paas-platform-as-a-service.md,
and the PR #3218 is to update existing file content/en/platform-as-a-service.md.

I suggest to close this PR and keep #3218.

@manojrayar
Copy link
Author

Hi @jihoon-seo , sure. I will do it.
Thanks.

@manojrayar manojrayar closed this Jul 23, 2024
@manojrayar
Copy link
Author

Closing this, have raised the PR #3218 for same.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lang/en for English
Projects
Development

Successfully merging this pull request may close these issues.

None yet

4 participants