pipeline-event-notification-rule: PipelineEventNotificationRule Should Accept id Parameter

[apogeeoak]

Describe the feature

PipelineEventNotificationRule should accept an id parameter instead of using a static string for the id.

Use Case

Creating two PipelineEventNotificationRule constructs in the same scope will throw an exception.

Proposed Solution

Update the constructor signature to include an id parameter to allow different PipelineEventNotificationRules to be uniquely identified.

Other Information

-  constructor(scope: CodePipeline, props: PipelineEventNotificationRuleProps) {
+  constructor(scope: CodePipeline, id: string, props: PipelineEventNotificationRuleProps) {

-    const targetTopic = Topic.fromTopicArn(scope, 'NotificationTopic', props.notificationTopicArn);
+    const targetTopic = Topic.fromTopicArn(scope, `NotificationTopic-${id}`, props.notificationTopicArn);

-    super(scope, `Pipeline-Event-Notification`, ruleProperties);
+    super(scope, `Pipeline-Event-Notification-${id}`, ruleProperties);

Acknowledgements

• I may be able to implement this feature request
• This feature might incur a breaking change

Library version used

0.6.2

Environment details (OS name and version, etc.)

N/A