Skip to content

Fix cut annotations JDT issue #103

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

Merged
merged 2 commits into from
Mar 14, 2025
Merged

Conversation

SentryMan
Copy link
Collaborator

@SentryMan SentryMan commented Mar 14, 2025

Not sure why, but it seems cut annotations can't handle single annotations with eclipse.

Checklist before merge

Have these changes been tested with the below to confirm no negative interactions?

  • avaje-inject
  • avaje-jsonb
  • avaje-validator
  • avaje-spi-service

Not sure why, but it seems cut annotations can't handle single annotations with eclipse
@SentryMan SentryMan added the bug Something isn't working label Mar 14, 2025
@SentryMan SentryMan self-assigned this Mar 14, 2025
@SentryMan SentryMan requested a review from rbygrave March 14, 2025 18:59
@SentryMan SentryMan enabled auto-merge (squash) March 14, 2025 19:02
SentryMan added a commit to avaje/avaje-validator that referenced this pull request Mar 14, 2025
@SentryMan
Copy link
Collaborator Author

I think it happens because jdt keeps the whitespace between annotation attributes

@SentryMan SentryMan merged commit 0397903 into avaje:main Mar 14, 2025
4 checks passed
@SentryMan SentryMan deleted the fix-cut-annotations branch March 14, 2025 23:32
rbygrave added a commit to avaje/avaje-validator that referenced this pull request Mar 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants