Skip to content

Rename supported to modelled.#2335

Merged
starcke merged 2 commits intomainfrom
starcke/rename-sup-modelled
Apr 17, 2023
Merged

Rename supported to modelled.#2335
starcke merged 2 commits intomainfrom
starcke/rename-sup-modelled

Conversation

@starcke
Copy link
Copy Markdown
Contributor

@starcke starcke commented Apr 17, 2023

I thought that using modelled instead of supported makes more sense when we start giving people a tool to model things themselves.

I didnt rename the query because the predicates matches what is in library, but we could consider that in the future.

Checklist

  • CHANGELOG.md has been updated to incorporate all user visible changes made by this pull request.
  • Issues have been created for any UI or other user-facing changes made by this pull request.
  • [Maintainers only] If this pull request makes user-facing changes that require documentation changes, open a corresponding docs pull request in the github/codeql repo and add the ready-for-doc-review label there.

@starcke starcke requested a review from koesie10 April 17, 2023 12:50
@starcke starcke requested a review from a team as a code owner April 17, 2023 12:50
Copy link
Copy Markdown
Contributor

@elenatanasoiu elenatanasoiu left a comment

Choose a reason for hiding this comment

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

Nice! I think modelled is clearer.

I've seen it written as "modeled" in documents. 🇺🇸 Do we want to be consistent with those?

@starcke
Copy link
Copy Markdown
Contributor Author

starcke commented Apr 17, 2023

I've seen it written as "modeled" in documents. us Do we want to be consistent with those?

Doh, yes!

@starcke
Copy link
Copy Markdown
Contributor Author

starcke commented Apr 17, 2023

Updated to use "modeled" instead. And also updated an old instance of "Unmodelled".

Copy link
Copy Markdown
Contributor

@elenatanasoiu elenatanasoiu left a comment

Choose a reason for hiding this comment

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

👍

@starcke starcke merged commit 0ee090b into main Apr 17, 2023
@starcke starcke deleted the starcke/rename-sup-modelled branch April 17, 2023 13:39
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.

2 participants