You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@devlake.apache.org by GitBox <gi...@apache.org> on 2022/09/06 09:34:45 UTC
[GitHub] [incubator-devlake] mappjzc opened a new issue, #2974: [Refactor][e2e] change tool table name to be const string
mappjzc opened a new issue, #2974:
URL: https://github.com/apache/incubator-devlake/issues/2974
## What and why to refactor
If we changed table name but foget to run e2e test before commit. it will create a new table and make the test result always be true.
So in @klesh opinion, it is better to use const string to set the tool table cvs file name.
## Describe the solution you'd like
refactor it as follow:
![image](https://user-images.githubusercontent.com/2921251/188599803-703e8118-6fe1-4083-a1e3-376dfd088e73.png)
## Related issues
Please link any other
## Additional context
Add any other context or screenshots about the feature request here.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@devlake.apache.org.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-devlake] klesh closed issue #2974: [Refactor][e2e] change tool table name to be const string
Posted by GitBox <gi...@apache.org>.
klesh closed issue #2974: [Refactor][e2e] change tool table name to be const string
URL: https://github.com/apache/incubator-devlake/issues/2974
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@devlake.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org