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

Update attachInterrupt.adoc #832

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Update attachInterrupt.adoc #832

wants to merge 1 commit into from

Conversation

krzky
Copy link

@krzky krzky commented May 27, 2021

more easy to find other

more easy to find other
@CLAassistant
Copy link

CLAassistant commented May 27, 2021

CLA assistant check
All committers have signed the CLA.

@@ -147,5 +147,8 @@ For Uno WiFiRev.2, Due, Zero, MKR Family and 101 boards the *interrupt number =
[float]
=== See also

[role="language"]
* #LANGUAGE# link:../../external-interrupts/detachinterrupt[detachInterrupts()]
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
* #LANGUAGE# link:../../external-interrupts/detachinterrupt[detachInterrupts()]
* #LANGUAGE# link:../../external-interrupts/detachinterrupt[detachInterrupt()]

Correct function name.

Comment on lines 152 to +153
--

Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
--
--

Use formatting style established by the reference sample:

Let's leave this part as it was because it is unrelated to the proposal and this is how the reference sample is formatted:

https://raw.githubusercontent.com/arduino/reference-en/master/AsciiDoc_sample/Reference_Terms/AsciiDoc_Template-Single_Entity.adoc

@per1234 per1234 added the bug label May 8, 2022
@per1234
Copy link
Collaborator

per1234 commented Jun 1, 2022

Hi @krzky. I wanted to check back in with you. If you will update the pull request according to the instructions I provided above I will be happy to merge this valuable enhancement.

@per1234 per1234 self-assigned this Aug 13, 2022
@per1234 per1234 added the on hold The PR should not be merged at this time label Aug 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug on hold The PR should not be merged at this time
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants