Skip to content

Conversation

@angusmcleod
Copy link
Member

No description provided.

@angusmcleod angusmcleod marked this pull request as draft February 14, 2022 06:32
@angusmcleod angusmcleod requested a review from fzngagan February 14, 2022 06:35
@lgtm-com
Copy link

lgtm-com bot commented Feb 14, 2022

This pull request introduces 1 alert when merging 7de00ca into 4605b23 - view on LGTM.com

new alerts:

  • 1 for Syntax error

@lgtm-com
Copy link

lgtm-com bot commented Feb 14, 2022

This pull request introduces 2 alerts when merging 5a347b6 into 4605b23 - view on LGTM.com

new alerts:

  • 1 for Unused variable, import, function or class
  • 1 for Invocation of non-function

@fzngagan
Copy link
Contributor

@angusmcleod
We're defining buildSubscriptionContent but calling generateSubscriptionContent

@angusmcleod angusmcleod marked this pull request as ready for review February 15, 2022 05:50
@fzngagan
Copy link
Contributor

@angusmcleod
Lets merge #182 into main and this one and make sure the correct branch of discourse and the plugin is checked out and the workflow succeeds.

Base automatically changed from pro-release to main September 23, 2022 15:47
@angusmcleod angusmcleod closed this Dec 5, 2023
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.

3 participants