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.

Sign up for GitHub

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

Jump to bottom

Harness ai rebrand #9508

Merged
merged 7 commits into from
Feb 27, 2025
Merged

Harness ai rebrand #9508

merged 7 commits into from
Feb 27, 2025

Conversation

vishal-av
Copy link
Collaborator

Thanks for contributing to the Harness Developer Hub! Our code owners will review your submission.

Description

  • Please describe your changes: __________________________________
  • Jira/GitHub Issue numbers (if any): ______________________________
  • Preview links/images (Internal contributors only): __________________

PR lifecycle

We aim to merge PRs within one week or less, but delays happen sometimes.

If your PR is open longer than two weeks without any human activity, please tag a code owner in a comment.

PRs must meet these requirements to be merged:

  • Successful preview build.
  • Code owner review.
  • No merge conflicts.
  • Release notes/new features docs: Feature/version released to at least one prod environment.

thisrohangupta and others added 3 commits February 20, 2025 11:53
@thisrohangupta
Update aida-overview.md
ef2c755
@thisrohangupta
Update aida-ci.md
6f0df28
@vishal-av
Changed keyword AIDA to Harness AI
d6e39a3 
Changed keyword AIDA to Harness AI as a part on ongoing  Harness AI rebranding
@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

✅ thisrohangupta
❌ vishal-av
You have signed the CLA already but the status is still pending? Let us recheck it.

@bot-gitexp-user
Copy link

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://67bd870dcb1ee4bc3d8827ad--harness-developer.netlify.app

@bot-gitexp-user
Copy link

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://67bd8dbdd7806e5bdba5d75f--harness-developer.netlify.app

ivikramsahu
ivikramsahu previously approved these changes Feb 25, 2025
View reviewed changes
Copy link
Collaborator

@ivikramsahu ivikramsahu left a comment

Choose a reason for hiding this comment

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

LGTM

@bot-gitexp-user
Copy link

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://67bec6b78d86d598ffd006f0--harness-developer.netlify.app

ivikramsahu
ivikramsahu approved these changes Feb 26, 2025
View reviewed changes
Copy link
Collaborator

@ivikramsahu ivikramsahu left a comment

Choose a reason for hiding this comment

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

LGTM

@dewan-ahmed dewan-ahmed self-requested a review February 26, 2025 12:56
pwolf-harness
pwolf-harness approved these changes Feb 26, 2025
View reviewed changes
Copy link
Collaborator

@pwolf-harness pwolf-harness left a comment

Choose a reason for hiding this comment

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

Lgtm

wei-harness
wei-harness approved these changes Feb 27, 2025
View reviewed changes
Copy link
Collaborator

@wei-harness wei-harness left a comment

Choose a reason for hiding this comment

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

LGTM, left a few comments about the names and slugs as well.

docs/cloud-cost-management/shared/ccm-aida-create-rule-partial.md
@@ -11,7 +11,7 @@ import TabItem from '@theme/TabItem';
<Tabs queryString="tab-number">
<TabItem value="4" label="Overview">

3. Select **Create a new rule with AIDA**.
3. Select **Create a new rule with AI**.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Do we replace AIDA with AI or Harness AI?

docs/cloud-cost-management/shared/ccm-aida-overview-partial.md
- [AIDA Terms](https://www.harness.io/legal/aida-terms)
- [AIDA Privacy](https://www.harness.io/legal/aida-privacy)
- [AI Terms](https://www.harness.io/legal/aida-terms)
- [AI Privacy](https://www.harness.io/legal/aida-privacy)
Copy link
Collaborator

Choose a reason for hiding this comment

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

Do the pathnames/slugs also need to be changed?

@vishal-av vishal-av merged commit d4a897b into main Feb 27, 2025
3 checks passed
@vishal-av vishal-av deleted the Harness-AI-Rebrand branch February 27, 2025 05:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@priteshchandaliya priteshchandaliya priteshchandaliya approved these changes

@dewan-ahmed dewan-ahmed dewan-ahmed approved these changes

@ivikramsahu ivikramsahu ivikramsahu approved these changes

@Rohit-Kaliki Rohit-Kaliki Rohit-Kaliki approved these changes

@thisrohangupta thisrohangupta thisrohangupta approved these changes

@nikunj-badjatya nikunj-badjatya nikunj-badjatya approved these changes

@wei-harness wei-harness wei-harness approved these changes

@nofarblue nofarblue nofarblue approved these changes

@shylajasun shylajasun shylajasun approved these changes

@krishi0408 krishi0408 krishi0408 approved these changes

@raviyadalam raviyadalam raviyadalam approved these changes

@SushrutHarness SushrutHarness SushrutHarness approved these changes

@pwolf-harness pwolf-harness pwolf-harness approved these changes

@pratmit pratmit Awaiting requested review from pratmit pratmit is a code owner

@GokulBansal0 GokulBansal0 Awaiting requested review from GokulBansal0 GokulBansal0 is a code owner

@abhishekharness abhishekharness Awaiting requested review from abhishekharness abhishekharness is a code owner

@navaneeth navaneeth Awaiting requested review from navaneeth navaneeth is a code owner

@joeyouss joeyouss Awaiting requested review from joeyouss joeyouss is a code owner

@iamanmolm iamanmolm Awaiting requested review from iamanmolm iamanmolm is a code owner

@sdsingh07 sdsingh07 Awaiting requested review from sdsingh07 sdsingh07 is a code owner

@sandyydk sandyydk Awaiting requested review from sandyydk sandyydk is a code owner

@abhinavsingh-ccm abhinavsingh-ccm Awaiting requested review from abhinavsingh-ccm abhinavsingh-ccm is a code owner

@tejakummarikuntla tejakummarikuntla Awaiting requested review from tejakummarikuntla tejakummarikuntla is a code owner

@hitesharinga hitesharinga Awaiting requested review from hitesharinga hitesharinga is a code owner

@rustd rustd Awaiting requested review from rustd rustd is a code owner

@shivloka-harness shivloka-harness Awaiting requested review from shivloka-harness shivloka-harness is a code owner

@shylajas shylajas Awaiting requested review from shylajas shylajas is a code owner

@rohanmaharjan100 rohanmaharjan100 Awaiting requested review from rohanmaharjan100 rohanmaharjan100 is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.