-
Notifications
You must be signed in to change notification settings - Fork 1
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
Design discussion for Online Importer and Exporter #3
Comments
Week 5 discussion:
|
Do we put any restrictions on the name of the behaviour? |
Hi @sudo-panda,
|
I would like to suggest a naming guideline:
Any other suggestions? |
Yes, the naming guideline looks good. |
Sir @pushkalkatara, sorry I don't understand. I can see an ExportText folder being created on the PR could you elaborate or recheck. |
Oops. I rechecked. Yes, the folder exists. We'll keep it that way only. |
Sirs, |
I have small suggestions for naming conventions. I think it is not a good practice to have space character in folder and file names. It is due to the inconvenience of command line operations. |
How did you implement threading, can you reference the source code? Did you try using QThreads? |
Week 4 discussion points:
The text was updated successfully, but these errors were encountered: