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

feat(annotations): add @CommandMethod annotation processing #366

Merged

Conversation

Citymonstret
Copy link
Member

We now verify the following at compile time:

  • That @CommandMethod annotated methods are non-static (error)
  • That @CommandMethod annotated methods are public (warning)
  • That the @CommandMethod syntax and specified @Arguments match
  • That no optional argument precedes a required argument

@Citymonstret Citymonstret enabled auto-merge (squash) May 31, 2022 18:18
@github-actions
Copy link

github-actions bot commented May 31, 2022

Unit Test Results

  45 files  +1    45 suites  +1   5s ⏱️ ±0s
203 tests +6  203 ✔️ +6  0 💤 ±0  0 ±0 
204 runs  +6  204 ✔️ +6  0 💤 ±0  0 ±0 

Results for commit 8db995b. ± Comparison against base commit 8a7dcd2.

♻️ This comment has been updated with latest results.

@Citymonstret Citymonstret force-pushed the feature/1.7.0/compile-time-commandmethod-validation branch from 3b8c8fc to 4eb57a9 Compare May 31, 2022 18:26
@Citymonstret Citymonstret added this to the 1.7.0 milestone May 31, 2022
@Citymonstret Citymonstret force-pushed the feature/1.7.0/compile-time-commandmethod-validation branch from 4eb57a9 to 99bc160 Compare May 31, 2022 18:44
We now verify the following at compile time:
- That `@CommandMethod` annotated methods are non-static (error)
- That `@CommandMethod` annotated methods are public (warning)
- That the `@CommandMethod` syntax and specified `@Argument`s match
- That no optional argument precedes a required argument
@Citymonstret Citymonstret force-pushed the feature/1.7.0/compile-time-commandmethod-validation branch from 99bc160 to 8db995b Compare May 31, 2022 18:58
@Citymonstret Citymonstret merged commit 1ff3b7a into 1.7.0-dev May 31, 2022
@Citymonstret Citymonstret deleted the feature/1.7.0/compile-time-commandmethod-validation branch May 31, 2022 19:06
Citymonstret added a commit that referenced this pull request Jun 3, 2022
We now verify the following at compile time:
- That `@CommandMethod` annotated methods are non-static (error)
- That `@CommandMethod` annotated methods are public (warning)
- That the `@CommandMethod` syntax and specified `@Argument`s match
- That no optional argument precedes a required argument
jpenilla pushed a commit that referenced this pull request Jun 19, 2022
We now verify the following at compile time:
- That `@CommandMethod` annotated methods are non-static (error)
- That `@CommandMethod` annotated methods are public (warning)
- That the `@CommandMethod` syntax and specified `@Argument`s match
- That no optional argument precedes a required argument
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