Skip to content
This repository has been archived by the owner on Jan 18, 2022. It is now read-only.

Support multiple annotations on method/type/enum blocks in Code Writer API #1427

Merged
merged 6 commits into from
Jul 20, 2020

Conversation

seanjparker
Copy link
Contributor

For UTY-2658

Description

Added support for adding multiple annotations in the Code Writer API on method/type/enum code blocks that use a ScopeBlock

Tests

How did you test these changes prior to submitting this pull request?
What automated tests are included in this PR?

  • Ran unit tests
  • Checked no errors on forced code generation

Documentation

How is this documented (for example: release note, upgrade guide, feature page, in-code documentation)?
Added Changelog entry

@improbable-prow-robot improbable-prow-robot added jira/no-ticket Indicates a PR has no corresponding JIRA ticket size/M Denotes a PR that changes 40-149 lines, ignoring generated files. labels Jul 15, 2020
@improbable-prow-robot improbable-prow-robot added the A: tooling Area: Tooling label Jul 15, 2020
@@ -7,17 +7,17 @@ namespace CodeGenerationLib.CodeWriter.Scopes
public class AnnotationOutsideType
{
protected readonly ScopeBlock ParentBlock;
protected readonly string Annotation;
protected readonly List<string> Annotations;
Copy link
Contributor

@BryanJY-Wong BryanJY-Wong Jul 20, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Missed this nevermind

Copy link
Contributor

@BryanJY-Wong BryanJY-Wong left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@sonarcloud
Copy link

sonarcloud bot commented Jul 20, 2020

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@seanjparker seanjparker merged commit a3f59d1 into develop Jul 20, 2020
@improbable-prow-robot improbable-prow-robot deleted the feature/codegen-multiple-annotations branch July 20, 2020 13:57
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A: tooling Area: Tooling jira/no-ticket Indicates a PR has no corresponding JIRA ticket size/M Denotes a PR that changes 40-149 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants