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

Adding Java header to parent pom.xml #1087

Merged
merged 1 commit into from
Jul 24, 2024
Merged

Conversation

artur-ciocanu
Copy link
Contributor

Description

This PR fixes a tiny issue with Checkstyle so we can run it from submodules as well. This was discovered by @salaboy.

Issue reference

We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.

Please reference the issue this PR will close: #1086

Checklist

Please make sure you've completed the relevant tasks for this PR, out of the following list:

  • Code compiles correctly
  • Created/updated tests
  • Extended the documentation

@artur-ciocanu artur-ciocanu requested review from a team as code owners July 22, 2024 14:41
@artur-ciocanu
Copy link
Contributor Author

@artursouza and @cicoyle this is another tiny PR that requires your attention. Thank you!

@artur-ciocanu
Copy link
Contributor Author

@artursouza and @cicoyle I was wondering is there a way to re-trigger the build? Sometimes we have IT tests failing because of container setup delays etc and I think it would be good to have a way to re-trigger the build without commits.

@artursouza
Copy link
Member

I do rerun them on GH actions. But GH only allows that with certain permissions, it is not open to all.

@artur-ciocanu
Copy link
Contributor Author

I do rerun them on GH actions. But GH only allows that with certain permissions, it is not open to all.

I see, makes sense. @artursouza if the PR looks good could you please merge?

Signed-off-by: Artur Ciocanu <ciocanu@adobe.com>
@artursouza artursouza merged commit 5618af5 into dapr:master Jul 24, 2024
7 checks passed
@artur-ciocanu artur-ciocanu deleted the gh-1086 branch July 24, 2024 18:44
salaboy pushed a commit to salaboy/java-sdk that referenced this pull request Jul 25, 2024
Signed-off-by: Artur Ciocanu <ciocanu@adobe.com>
Co-authored-by: Artur Ciocanu <ciocanu@adobe.com>
Signed-off-by: salaboy <Salaboy@gmail.com>
salaboy pushed a commit to salaboy/java-sdk that referenced this pull request Jul 26, 2024
Signed-off-by: Artur Ciocanu <ciocanu@adobe.com>
Co-authored-by: Artur Ciocanu <ciocanu@adobe.com>
Signed-off-by: salaboy <Salaboy@gmail.com>
salaboy pushed a commit to salaboy/java-sdk that referenced this pull request Jul 26, 2024
Signed-off-by: Artur Ciocanu <ciocanu@adobe.com>
Co-authored-by: Artur Ciocanu <ciocanu@adobe.com>
Signed-off-by: salaboy <Salaboy@gmail.com>
salaboy pushed a commit to salaboy/java-sdk that referenced this pull request Jul 26, 2024
Signed-off-by: Artur Ciocanu <ciocanu@adobe.com>
Co-authored-by: Artur Ciocanu <ciocanu@adobe.com>
Signed-off-by: salaboy <Salaboy@gmail.com>
salaboy pushed a commit to salaboy/java-sdk that referenced this pull request Jul 26, 2024
Signed-off-by: Artur Ciocanu <ciocanu@adobe.com>
Co-authored-by: Artur Ciocanu <ciocanu@adobe.com>
Signed-off-by: salaboy <Salaboy@gmail.com>
salaboy pushed a commit to salaboy/java-sdk that referenced this pull request Jul 26, 2024
Signed-off-by: Artur Ciocanu <ciocanu@adobe.com>
Co-authored-by: Artur Ciocanu <ciocanu@adobe.com>
Signed-off-by: salaboy <Salaboy@gmail.com>
salaboy pushed a commit to salaboy/java-sdk that referenced this pull request Jul 26, 2024
Signed-off-by: Artur Ciocanu <ciocanu@adobe.com>
Co-authored-by: Artur Ciocanu <ciocanu@adobe.com>
Signed-off-by: salaboy <Salaboy@gmail.com>
salaboy pushed a commit to salaboy/java-sdk that referenced this pull request Jul 27, 2024
Signed-off-by: Artur Ciocanu <ciocanu@adobe.com>
Co-authored-by: Artur Ciocanu <ciocanu@adobe.com>
Signed-off-by: salaboy <Salaboy@gmail.com>
salaboy pushed a commit to salaboy/java-sdk that referenced this pull request Jul 27, 2024
Signed-off-by: Artur Ciocanu <ciocanu@adobe.com>
Co-authored-by: Artur Ciocanu <ciocanu@adobe.com>
Signed-off-by: salaboy <Salaboy@gmail.com>
salaboy pushed a commit to salaboy/java-sdk that referenced this pull request Jul 29, 2024
Signed-off-by: Artur Ciocanu <ciocanu@adobe.com>
Co-authored-by: Artur Ciocanu <ciocanu@adobe.com>
Signed-off-by: salaboy <Salaboy@gmail.com>
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.

Checkstyle can't be run from submodules
2 participants