Skip to content

Changes and minor refactoring to make deployments work on Linux. #1

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

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

Conversation

mmkathurima
Copy link

Included:

  • Changed a constant referencing the new line character to a cross platform character i.e. Environment.NewLine. Using vbCrLf works on Windows. In this code, it is used in the IndexOf and LastIndexOf methods. The code as it is fails to consider the differences in line terminators across OSes and locating indices fails on non-Windows environments.
  • Addition of explicit types during variable declaration.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant