cqframework / cqframework/cqf-tooling

Replace path delimiter with File.separator

Open
#187 1 comment 0 reactions 1 assignee View on GitHub

@Chris0296 is already working on this.

Since Sep 20, 2023.

bug
Dominant language
Java
Stars
22
Forks
26
PR merge metrics
No merged PRs in 30d

Description

Describe the bug
Throughout the project "/" is used as a delimiter for file paths. This is OS specific. The "/" is for Linux/Mac and the "" is for Windows. We need to replace the delimiter with File.separator which is not OS specific. I replaced it in BundleResources.java. We need to be careful to differentiate between url paths and file paths.

To Reproduce
Search the java files for "/"

Expected behavior
File paths will work regardless of the OS.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.