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

Apply project settings to o.e.equinox.app #323

Merged

Conversation

Torbjorn-Svensson
Copy link
Contributor

@Torbjorn-Svensson Torbjorn-Svensson commented Oct 1, 2023

  1. Convert to LF and ensure LF at EOF:
find . -type f -print0 | \
  xargs -r0 perl -le 'for (@argv) { print if -f && -T }' | \
  xargs -rd'\n' dos2unix -e && \
  git checkout osgi/org/osgi
  1. Format java files:
eclipse -consolelog -nosplash -application org.eclipse.jdt.core.JavaCodeFormatter \
  -config .settings/org.eclipse.jdt.core.prefs . -data `mktemp -d` && \
  git checkout osgi/org/osgi

@github-actions
Copy link

github-actions bot commented Oct 1, 2023

Test Results

     24 files  ±0       24 suites  ±0   11m 55s ⏱️ +24s
2 150 tests ±0  2 104 ✔️ ±0  45 💤 ±0  1 ±0 
2 194 runs  ±0  2 148 ✔️ ±0  45 💤 ±0  1 ±0 

For more details on these failures, see this check.

Results for commit 5081f63. ± Comparison against base commit 8beb9ce.

♻️ This comment has been updated with latest results.

@Torbjorn-Svensson Torbjorn-Svensson force-pushed the pr/cleanup-o.e.equinox.app branch 2 times, most recently from efece83 to 4855a47 Compare October 5, 2023 18:02
@Torbjorn-Svensson
Copy link
Contributor Author

Updated PR in accordance with #324 (comment).

This was achieved by running:
find . -type f -print0 | \
  xargs -r0 perl -le 'for (@argv) { print if -f && -T }' | \
  xargs -rd'\n' dos2unix -e && \
  git checkout osgi/org/osgi

Signed-off-by: Torbjörn SVENSSON <torbjorn.svensson@st.com>
This was achieved by running:
eclipse -consolelog -nosplash -application org.eclipse.jdt.core.JavaCodeFormatter \
  -config .settings/org.eclipse.jdt.core.prefs . -data `mktemp -d` && \
  git checkout osgi/org/osgi

Signed-off-by: Torbjörn SVENSSON <torbjorn.svensson@st.com>
@Torbjorn-Svensson Torbjorn-Svensson force-pushed the pr/cleanup-o.e.equinox.app branch from 4855a47 to 5081f63 Compare October 27, 2023 14:37
@akurtakov akurtakov merged commit 8da08ff into eclipse-equinox:master Nov 3, 2023
19 of 23 checks passed
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.

2 participants