incubator-answer
incubator-answer copied to clipboard
Master Update
Languages
- Pronunciation optimised
Frontend
- Use NodeJS 18 and higher
- Removed pnpm force
- Updated all NodeJS dependencies for the frontend to latest version
- No link to the GitHub release page of Answer, if version already up to date
- Version check is skipped if Answer was installed directly from GitHub
Backend
- Allowing of uppercase usernames
- Allowing of shorter/longer (user)names, passwords, mails and site names
- Allowing of shorter/longer object ids
- Allowing of shorter/longer new/old revision ids
- Allowing of shorter/longer querys
- Allowing of shorter/longer plugin slug names
- Allowing of shorter/longer report contents
- Allowing of shorter/longer Question titles, contents and ids
First of all, I want to talk about how to make a better contribution. From a developer's point of view, each PR should be clear. The current PR solves a specific problem or completes a complete function; if you If there are multiple features or Isuu that need to be contributed, they should be split into multiple PRs to submit, so that there will be no additional burden when others come to Reiview.
Secondly, when changing some project configurations or basic environments, a discussion needs to be initiated first, and then community users will decide whether these configurations or environments need to be changed.
Use NodeJS 18 and higher
We only need the minimum version to ensure the normal operation of the project
Removed pnpm force
We started with pnpm for a few reasons
- pnpm has better performance and memory management than yarn or npm.
- our CI also uses pnpm and our team uses pnpm as the front-end package management tool.
- for developers or contributors, it is up to them to decide which package management tool to use. They just need to make sure not to submit these files when submitting a PR.
updated all NodeJS dependencies for the frontend to latest version
whether dependencies need to be upgraded, we will decide based on the security report and whether the existing dependencies meet the functions of the current version. Each dependency upgrade requires a lot of testing, so a certain dependency needs to be upgraded. Please explain the reason for the upgrade, and we will update it. Determine whether to upgrade based on specific circumstances
No link to the GitHub release page of Answer, if version already up to date
If need to do this, please go to ISSU to initiate a discussion. We will decide whether adjustments are needed based on user feedback.
Version check is skipped if Answer was installed directly from GitHub
Reference here #764
Finally, thank you very much for your contribution. If you have any questions, you can continue the discussion.
First of all, I want to talk about how to make a better contribution. From a developer's point of view, each PR should be clear. The current PR solves a specific problem or completes a complete function; if you If there are multiple features or Isuu that need to be contributed, they should be split into multiple PRs to submit, so that there will be no additional burden when others come to Reiview.
Secondly, when changing some project configurations or basic environments, a discussion needs to be initiated first, and then community users will decide whether these configurations or environments need to be changed.
Use NodeJS 18 and higher
We only need the minimum version to ensure the normal operation of the project
Removed pnpm force
We started with pnpm for a few reasons
1. pnpm has better performance and memory management than yarn or npm. 2. our CI also uses pnpm and our team uses pnpm as the front-end package management tool. 3. for developers or contributors, it is up to them to decide which package management tool to use. They just need to make sure not to submit these files when submitting a PR.
updated all NodeJS dependencies for the frontend to latest version
whether dependencies need to be upgraded, we will decide based on the security report and whether the existing dependencies meet the functions of the current version. Each dependency upgrade requires a lot of testing, so a certain dependency needs to be upgraded. Please explain the reason for the upgrade, and we will update it. Determine whether to upgrade based on specific circumstances
No link to the GitHub release page of Answer, if version already up to date
If need to do this, please go to ISSU to initiate a discussion. We will decide whether adjustments are needed based on user feedback.
Version check is skipped if Answer was installed directly from GitHub
Reference here #764
Finally, thank you very much for your contribution. If you have any questions, you can continue the discussion.
- So, one cannot use capital letters during the installation, which is definitely not the way it should be! The database doesn't care if the username is uppercase or lowercase!
- Updating dependencies only when it's needed is the stupidest setting you can have! Nobody updates their dependencies only when it's needed! One always does it, even if there are no security risks
- In the script to build the plugins, the sleep statements are intentional, because one can only read the text that is output by echo before the script bombs the console, and one can no longer read it properly
- Do I have to initialise the TryToInstallByEnv again so that the variable
installByEnv
gets a value? - The tag check when asking questions is superfluous, as one does not need it to be able to ask questions (a tag force is also quite bold)
Furthermore, I test all my changes before I push them! So all my commits can be merged without hesitation
And if I were to discuss all my changes that I want to implement first, I could just forget about it, because everyone else could then simply commit these changes themselves because they would then know about them
@shuashuai @LinkinStars
Please split according to the improvement suggestions and resubmit! The current PR was discussed as not meeting the requirements.