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

Update project name throughout the repository #234

Merged
merged 10 commits into from
Oct 7, 2021

Conversation

lbianchi-lbl
Copy link
Contributor

Summary/Motivation

  • Replace project name everywhere so that everything works with new project name

Legal Acknowledgement

By contributing to this software project, I agree to the following terms and conditions for my contribution:

  1. I agree my contributions are submitted under the license terms described in the LICENSE.txt file at the top level of this directory.
  2. I represent I am authorized to make the contributions and grant the license. If my employer has rights to intellectual property that includes these contributions, I represent that I have received permission to make contributions and grant the required license on behalf of that employer.

@lbianchi-lbl lbianchi-lbl added the Priority:High High Priority Issue or PR label Oct 7, 2021
@lbianchi-lbl
Copy link
Contributor Author

I needed to work a bit to set up ReadTheDocs, but it looks like it's working now (even though we might have to wait until this PR is merged in to have the PR check show up in the PR Checks menu): https://watertap--234.org.readthedocs.build/en/234/index.html.

If you don't have anything against, I'd say we can go ahead and merge this in, and deal with any remaining fixes in separate PRs.

Copy link
Contributor

@bknueven bknueven left a comment

Choose a reason for hiding this comment

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

The tests pass, and a cursory git grep for proteuslib and related names comes up empty locally.

@lbianchi-lbl lbianchi-lbl enabled auto-merge (squash) October 7, 2021 23:31
Copy link
Contributor

@ksbeattie ksbeattie left a comment

Choose a reason for hiding this comment

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

LGTM

@lbianchi-lbl lbianchi-lbl merged commit 1a7aaa4 into watertap-org:main Oct 7, 2021
@ksbeattie ksbeattie added this to In progress in 2021 Dec (Q8) Release via automation Oct 8, 2021
@ksbeattie ksbeattie moved this from In progress to Done in 2021 Dec (Q8) Release Oct 8, 2021
lbianchi-lbl added a commit to watertap-org/parameter-sweep that referenced this pull request May 1, 2024
)

* Rename project name ProteusLib -> WaterTAP

* Rename GitHub full name: nawi-hub/proteuslib -> watertap-org/watertap

* Rename Python package name in .py files: proteuslib -> watertap

* Rename Python package name in .rst files: proteuslib -> watertap

* Rename Python package name in other files: proteuslib -> watertap

* Rename directories and files: proteuslib -> watertap

* Rename project slug in docs links: proteuslib -> watertap

* Rename project name (alternate capitalization): Proteuslib -> WaterTAP

* Whitespace commit to trigger RTD build on PR

* Rename ReadTheDocs project slug: proteuslib -> watertap
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Priority:High High Priority Issue or PR
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

3 participants