Skip to content
This repository has been archived by the owner on Jun 17, 2019. It is now read-only.

Changed Task Generation to be Cross Platform #34

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Changed Task Generation to be Cross Platform #34

wants to merge 1 commit into from

Conversation

mikaelweave
Copy link

After trying to run the generator on my Arch Linux WSL environment, I saw that there were some files that were failing to be copied (breaking the whole process) due to mismatch from file and config capitalization. This isn't an issue in Windows due to the file system being case insensitive, but is an issue in Linux.

Main problem with the Task Generator was the capitalization of the "tasks" source folder when copying files to the generator output folder. There was also a mismatch for the screenshot file.

The problems with the tests was both capitalization and file paths being Windows specific.

Also modified the taskTests to be more OS safe. Windows doesn't seem to care about paths with "/" but Linux/Node does not agree with "" in paths.

Please let me know if this type of work is appreciated and I can look at the rest of the generators. ❤️

@ALM-Rangers/generator-vsts-extension

After trying to run the generator on my Arch Linux
WSL environment, I saw that there were some files
that were failing to be copied (breaking the
whole process) due to capitalization.

Also modified the tastTests to be more OS
safe. Windows doesn't seem to care about
paths with "/" but Arch hates "\" in paths
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant