Skip to content

build(aio): render whether API classes and members are abstract#22563

Closed
petebacondarwin wants to merge 2 commits intoangular:masterfrom
petebacondarwin:aio-render-abstract
Closed

build(aio): render whether API classes and members are abstract#22563
petebacondarwin wants to merge 2 commits intoangular:masterfrom
petebacondarwin:aio-render-abstract

Conversation

@petebacondarwin
Copy link
Contributor

Closes #22537

@petebacondarwin petebacondarwin added action: review The PR is still awaiting reviews from at least one requested reviewer comp: aio target: major This PR is targeted for the next major release labels Mar 2, 2018
@mary-poppins
Copy link

You can preview 646cde8 at https://pr22563-646cde8.ngbuilds.io/.

{%- macro renderMemberSyntax(member, truncateLines) -%}
{%- if member.accessibility !== 'public' %}{$ member.accessibility $} {% endif -%}
{%- if member.isAbstract %}abstract {% endif -%}
{%- if member.isStatic %}static {% endif -%}
Copy link
Member

@gkalpak gkalpak Mar 3, 2018

Choose a reason for hiding this comment

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

Optional nit: Extra space after isStatic 🙄

@gkalpak gkalpak added action: cleanup The PR is in need of cleanup, either due to needing a rebase or in response to comments from reviews and removed action: review The PR is still awaiting reviews from at least one requested reviewer labels Mar 3, 2018
@petebacondarwin petebacondarwin added action: merge The PR is ready for merge by the caretaker and removed action: cleanup The PR is in need of cleanup, either due to needing a rebase or in response to comments from reviews labels Mar 3, 2018
@mary-poppins
Copy link

You can preview a672065 at https://pr22563-a672065.ngbuilds.io/.

@petebacondarwin
Copy link
Contributor Author

petebacondarwin commented Mar 3, 2018

@alexeagle alexeagle closed this in b80fd6b Mar 5, 2018
@petebacondarwin petebacondarwin deleted the aio-render-abstract branch March 7, 2018 18:45
@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Sep 13, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

action: merge The PR is ready for merge by the caretaker cla: yes target: major This PR is targeted for the next major release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

aio + api/docs - render abstract modifiers on abstract classes and members

4 participants