Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Use new static IPath factories and constants and only the IPath type #516

Merged
merged 1 commit into from
Jul 10, 2023

Conversation

HannesWell
Copy link
Member

@HannesWell HannesWell commented Jun 24, 2023

Use the new IPath factory methods introduced in eclipse-equinox/equinox#228 in Eclipse-Platform.

@github-actions
Copy link
Contributor

github-actions bot commented Jun 24, 2023

Test Results

       42 files  ±0         42 suites  ±0   53m 31s ⏱️ - 2m 26s
  3 754 tests ±0    3 750 ✔️ +1    4 💤 ±0  0 ±0 
11 265 runs  ±0  11 235 ✔️ +1  30 💤 ±0  0 ±0 

Results for commit 2021f73. ± Comparison against base commit c5416ba.

♻️ This comment has been updated with latest results.

@HannesWell HannesWell force-pushed the ipathFactories branch 2 times, most recently from d1ce0d3 to cf57666 Compare June 25, 2023 13:15
@HannesWell HannesWell changed the title Use new static IPath factories and constants Use new static IPath factories and constants and only the IPath type Jun 25, 2023
@HannesWell HannesWell force-pushed the ipathFactories branch 4 times, most recently from e355d8f to 395c74c Compare July 8, 2023 11:28
Apply the following replacements:
- new Path(String) -> IPath.fromOSString(String)
- Path.fromOSString/fromPortableString/forPosix/forWindows(String)
  -> IPath.fromOSString/fromPortableString/forPosix/forWindows(String)
@HannesWell HannesWell merged commit 26abe91 into eclipse-platform:master Jul 10, 2023
12 checks passed
@HannesWell HannesWell deleted the ipathFactories branch July 10, 2023 06:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant