check-i18n.pl is linked out of directory #11247
Comments
|
The project should not have the react-sdk within the riot-web directory, so the symlink is correct (if one were to follow the development instructions). Realistically it probably shouldn't be a symlink, but the alternative is code duplication :( |
Could it not expose it like /~https://github.com/matrix-org/matrix-react-sdk/blob/develop/package.json#L32-L36 |
|
There was a reason why we didn't do that, but maybe yarn fixes it |
|
@turt2live Oh, I see. The directions from build from source end up with the folders inside riot web, while the development instructions don't. Maybe we should consider changing that to make them consistant? Could be as simple as having |
|
It does seem like we're creating a trap for people by having I guess the tricky part is |
|
We shouldn't be encouraging people to use the CI scripts. If I have to rename that script to stop people using it, I will :| The development instructions should be making no reference to that script. |
|
Let's just remove references to that script from the docs then. |
|
Renaming seems like a good idea, |
This removes `fetch-develop.deps.sh` from the docs, as it's designed more for use on CI environments and ends up causing confusion when used for anything else. Fixes #11247
Description
The file
scripts/check-i18n.plis a symbolic link to../../matrix-react-sdk/scripts/check-i18n.pl. If I understand the project set up, we can't rely on that file existing there, it instead should link to../matrix-react-sdk/scripts/check-i18n.pl. This doesn't seem to be causing problems with normal use, but can break some tools i.e. trying to use grep inside the project folder will break.I can fix this really quick if someone can confirm its an issue and not me misunderstanding something.
Steps to reproduce
ls -l scripts/check-i18n.plSymbolic links should point to files that exist.
Logs being sent: no
Version information
N/A
Centos 7
The text was updated successfully, but these errors were encountered: