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

Rename non-expressive variables #8521

Closed
ericoBandeira opened this issue Apr 22, 2021 · 4 comments
Closed

Rename non-expressive variables #8521

ericoBandeira opened this issue Apr 22, 2021 · 4 comments
Labels
area:rasa-oss 🎡 Anything related to the open source Rasa framework type:maintenance 🔧 Improvements to tooling, testing, deployments, infrastructure, code style.

Comments

@ericoBandeira
Copy link
Contributor

ericoBandeira commented Apr 22, 2021

Description of Problem: on the files:

  • .../rasa/core/tracker_store.py ;
  • .../rasa/utils/io.py;
  • /rasa/core/channels/mattermost.py.

has some variables with a non expressive name.

Overview of the Solution: Changing variables that are just letters to names that express exactly what the variable represents.

@ericoBandeira ericoBandeira added area:rasa-oss 🎡 Anything related to the open source Rasa framework type:enhancement ✨ Additions of new features or changes to existing ones, should be doable in a single PR labels Apr 22, 2021
@sara-tagger
Copy link
Collaborator

sara-tagger commented Apr 22, 2021

Exalate commented:

sara-tagger commented:

Thanks for submitting this feature request

🚀
@koernerfelicia will get back to you about it soon!


@koernerfelicia
Copy link
Contributor

koernerfelicia commented Apr 22, 2021

Exalate commented:

koernerfelicia commented:

Hi @ericoBandeira I see you've already linked a pull request!

Thank you for that, my colleague will take a look at it. Please be sure to take care of the failing test – looks like you need to reformat the file. Read more about how to do this locally here.

@TyDunn TyDunn added type:maintenance 🔧 Improvements to tooling, testing, deployments, infrastructure, code style. and removed type:enhancement ✨ Additions of new features or changes to existing ones, should be doable in a single PR labels May 26, 2021
@stale
Copy link

stale bot commented Jan 9, 2022

Exalate commented:

stale[bot] commented:

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label Jan 9, 2022
@TyDunn TyDunn removed the stale label Jan 27, 2022
@rasabot-exalate rasabot-exalate added area:rasa-oss and removed type:maintenance 🔧 Improvements to tooling, testing, deployments, infrastructure, code style. area:rasa-oss 🎡 Anything related to the open source Rasa framework labels Mar 15, 2022 — with Exalate Issue Sync
@m-vdb m-vdb added type:maintenance 🔧 Improvements to tooling, testing, deployments, infrastructure, code style. area:rasa-oss 🎡 Anything related to the open source Rasa framework and removed type:maintenance labels Mar 16, 2022
@rasabot-exalate rasabot-exalate added area:rasa-oss_:ferris_wheel: area:rasa-oss 🎡 Anything related to the open source Rasa framework type:maintenance 🔧 Improvements to tooling, testing, deployments, infrastructure, code style. and removed type:maintenance 🔧 Improvements to tooling, testing, deployments, infrastructure, code style. area:rasa-oss 🎡 Anything related to the open source Rasa framework area:rasa-oss_:ferris_wheel: labels Mar 17, 2022 — with Exalate Issue Sync
@sync-by-unito
Copy link

sync-by-unito bot commented Dec 19, 2022

➤ Maxime Verger commented:

💡 Heads up! We're moving issues to Jira: https://rasa-open-source.atlassian.net/browse/OSS.

From now on, this Jira board is the place where you can browse (without an account) and create issues (you'll need a free Jira account for that). This GitHub issue has already been migrated to Jira and will be closed on January 9th, 2023. Do not forget to subscribe to the corresponding Jira issue!

➡️ More information in the forum: https://forum.rasa.com/t/migration-of-rasa-oss-issues-to-jira/56569.

@m-vdb m-vdb closed this as completed Jan 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:rasa-oss 🎡 Anything related to the open source Rasa framework type:maintenance 🔧 Improvements to tooling, testing, deployments, infrastructure, code style.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants