Skip to content

SG-34087 - Rebranding#321

Merged
carlos-villavicencio-adsk merged 8 commits into
masterfrom
ticket/SG-34087-rebranding
Mar 20, 2024
Merged

SG-34087 - Rebranding#321
carlos-villavicencio-adsk merged 8 commits into
masterfrom
ticket/SG-34087-rebranding

Conversation

@juanburgosautoglb
Copy link
Copy Markdown
Contributor

Change all the shotgrid/shotgun occurrences in the repository.

Comment thread HISTORY.rst Outdated
Comment thread README.md Outdated
Comment thread SECURITY.md
Copy link
Copy Markdown
Member

@julien-lang julien-lang left a comment

Choose a reason for hiding this comment

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

Please make sure to respect the line width rule when exists in the file

Comment thread README.md Outdated
Comment thread docs/authentication.rst Outdated
Comment thread docs/advanced/packaging.rst Outdated
Comment thread docs/cookbook/examples/svn_integration.rst Outdated
Comment thread docs/index.rst
Comment thread shotgun_api3/shotgun.py Outdated
Comment thread tests/example_config Outdated
Copy link
Copy Markdown
Member

@julien-lang julien-lang left a comment

Choose a reason for hiding this comment

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

Please check last changes. I see a lof of "eaten spaces"

Comment thread shotgun_api3/shotgun.py Outdated
Copy link
Copy Markdown
Member

@julien-lang julien-lang left a comment

Choose a reason for hiding this comment

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

A few minor comments. We are almost there.

Comment thread HISTORY.rst Outdated
Comment thread HISTORY.rst Outdated
Comment thread docs/cookbook.rst Outdated
Comment thread docs/cookbook/smart_cut_fields.rst Outdated
@juanburgosautoglb juanburgosautoglb force-pushed the ticket/SG-34087-rebranding branch from ab0aeb2 to 5212f8a Compare February 8, 2024 15:47
Copy link
Copy Markdown
Member

@julien-lang julien-lang left a comment

Choose a reason for hiding this comment

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

Looks good. I checked the branch and I still see 2 references to ShotGrid:

  • setup.py
  • software_credits

Copy link
Copy Markdown
Member

@julien-lang julien-lang left a comment

Choose a reason for hiding this comment

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

LGTM!

@carlos-villavicencio-adsk carlos-villavicencio-adsk merged commit 94718c4 into master Mar 20, 2024
@carlos-villavicencio-adsk carlos-villavicencio-adsk deleted the ticket/SG-34087-rebranding branch March 20, 2024 16:34
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