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

Formatting markdown files #127

Merged
merged 24 commits into from
Jul 29, 2024
Merged

Formatting markdown files #127

merged 24 commits into from
Jul 29, 2024

Conversation

DennisDyallo
Copy link
Collaborator

@DennisDyallo DennisDyallo commented Jul 12, 2024

Description

Formatting md-files, applying proper format of tables,
as well as adding them to the solution

Copy link

github-actions bot commented Jul 12, 2024

Test Results: Windows

    2 files      2 suites   4s ⏱️
3 700 tests 3 700 ✅ 0 💤 0 ❌
3 702 runs  3 702 ✅ 0 💤 0 ❌

Results for commit dc6ed27.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented Jul 12, 2024

Test Results: Ubuntu

    2 files      2 suites   9s ⏱️
3 692 tests 3 692 ✅ 0 💤 0 ❌
3 694 runs  3 694 ✅ 0 💤 0 ❌

Results for commit dc6ed27.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented Jul 12, 2024

Test Results: MacOS

    2 files      2 suites   3s ⏱️
3 692 tests 3 692 ✅ 0 💤 0 ❌
3 694 runs  3 694 ✅ 0 💤 0 ❌

Results for commit dc6ed27.

♻️ This comment has been updated with latest results.

@DennisDyallo DennisDyallo requested a review from equijano21 July 12, 2024 11:22
@DennisDyallo
Copy link
Collaborator Author

Ran the IDE code/cleanup formatter on the MD files as well as manually fixed the tables. Feel free to merge when you've glanced through it @equijano21 . Also added your latest changes from the #126 PR

Data | SW1 | SW2
:---: | :---: | :---:
Data | SW1 | SW2

Copy link
Collaborator

Choose a reason for hiding this comment

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

extra line causing problems

Data | SW1 | SW2
:---: | :---: | :---:
Data | SW1 | SW2

Copy link
Collaborator

Choose a reason for hiding this comment

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

extra line causing problems

Copy link
Collaborator

Choose a reason for hiding this comment

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

File not in toc.

Copy link
Collaborator

Choose a reason for hiding this comment

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

File not in toc.

Copy link
Collaborator

Choose a reason for hiding this comment

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

File not in toc.

> If a YubiKey has been configured to require a button touch, you must make sure to alert the user of this requirement during a challenge-response operation. This can be accomplished by calling the [UseTouchNotifier()](xref:Yubico.YubiKey.Otp.Operations.CalculateChallengeResponse.UseTouchNotifier(System.Action)) method when sending a challenge to a YubiKey via ``CalculateChallengeResponse()``.
> If a YubiKey has been configured to require a button touch, you must make sure to alert the user of this requirement
> during a challenge-response operation. This can be accomplished by calling
>
Copy link
Collaborator

Choose a reason for hiding this comment

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

Extra line causing formatting issues.


To enable the manual update feature, you must:

1. set the [static ticket flag](xref:Yubico.YubiKey.Otp.ConfigurationFlags.StaticTicket), then

1. set [AllowManualUpdate()](xref:Yubico.YubiKey.Otp.Operations.ConfigureStaticPassword.AllowManualUpdate%28System.Boolean%29) to ``true`` when calling ``ConfigureStaticPassword()``.
1.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Extra line causing issues.

Copy link
Collaborator

Choose a reason for hiding this comment

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

File not in toc

@@ -43,31 +50,37 @@ The rest of the input data is related to the new credential:
Further information is found in the [Data](#data) section.

> [!NOTE]
> The label may only contain characters that can be encoded with UTF-8, and its UTF-8 byte count must be between 1 and 64. Non-printing characters are allowed, as long as they can be encoded with UTF-8. For example, null (UTF-8: 0x00) and Right-To-Left Mark U+200F (UTF-8: 0xE2 0x80 0x8F) would be accepted. Since the label is used for display purposes, it is recommended to prefer printable characters.
> The label may only contain characters that can be encoded with UTF-8, and its UTF-8 byte count must be between 1 and

Copy link
Collaborator

Choose a reason for hiding this comment

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

Extra line causing formatting issues.

@@ -21,7 +21,10 @@ Store long-lived keys in the YubiHSM Auth application by creating a new [credent

All YubiKeys with the YubiHSM Auth application (included in firmware version 5.4.3 and later).
> [!NOTE]
> Use the .NET API's [HasFeature()](xref:Yubico.YubiKey.YubiKeyFeatureExtensions.HasFeature%28Yubico.YubiKey.IYubiKeyDevice%2CYubico.YubiKey.YubiKeyFeature%29) method to check if a key has the YubiHSM Auth application.
> Use the .NET
>
Copy link
Collaborator

Choose a reason for hiding this comment

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

Extra line causing formatting issues.


## Available

All YubiKeys with the YubiHSM Auth application (included in firmware version 5.4.3 and later).
> [!NOTE]
> Use the .NET API's [HasFeature()](xref:Yubico.YubiKey.YubiKeyFeatureExtensions.HasFeature%28Yubico.YubiKey.IYubiKeyDevice%2CYubico.YubiKey.YubiKeyFeature%29) method to check if a key has the YubiHSM Auth application.
> Use the .NET
>
Copy link
Collaborator

Choose a reason for hiding this comment

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

Extra line causing formatting issues.

@@ -24,7 +24,10 @@ Remove a credential from the YubiHSM Auth application.

All YubiKeys with the YubiHSM Auth application (included in firmware version 5.4.3 and later).
> [!NOTE]
> Use the .NET API's [HasFeature()](xref:Yubico.YubiKey.YubiKeyFeatureExtensions.HasFeature%28Yubico.YubiKey.IYubiKeyDevice%2CYubico.YubiKey.YubiKeyFeature%29) method to check if a key has the YubiHSM Auth application.
> Use the .NET
>
Copy link
Collaborator

Choose a reason for hiding this comment

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

Extra line causing formatting issues.

@@ -24,7 +24,10 @@ Get the SCP03 session keys from an AES-128 credential.

All YubiKeys with the YubiHSM Auth application (included in firmware version 5.4.3 and later).
> [!NOTE]
> Use the .NET API's [HasFeature()](xref:Yubico.YubiKey.YubiKeyFeatureExtensions.HasFeature%28Yubico.YubiKey.IYubiKeyDevice%2CYubico.YubiKey.YubiKeyFeature%29) method to check if a key has the YubiHSM Auth application.
> Use the .NET
>
Copy link
Collaborator

Choose a reason for hiding this comment

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

Extra line causing formatting issues.

@@ -24,7 +24,10 @@ Get the version of the YubiHSM Auth application returned as a major, minor, and

All YubiKeys with the YubiHSM Auth application (included in firmware version 5.4.3 and later).
> [!NOTE]
> Use the .NET API's [HasFeature()](xref:Yubico.YubiKey.YubiKeyFeatureExtensions.HasFeature%28Yubico.YubiKey.IYubiKeyDevice%2CYubico.YubiKey.YubiKeyFeature%29) method to check if a key has the YubiHSM Auth application.
> Use the .NET
>
Copy link
Collaborator

Choose a reason for hiding this comment

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

Extra line causing formatting issues.


## Available

All YubiKeys with the YubiHSM Auth application (included in firmware version 5.4.3 and later).
> [!NOTE]
> Use the .NET API's [HasFeature()](xref:Yubico.YubiKey.YubiKeyFeatureExtensions.HasFeature%28Yubico.YubiKey.IYubiKeyDevice%2CYubico.YubiKey.YubiKeyFeature%29) method to check if a key has the YubiHSM Auth application.
> Use the .NET
>
Copy link
Collaborator

Choose a reason for hiding this comment

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

Extra line causing formatting issues.


## Available

All YubiKeys with the YubiHSM Auth application (included in firmware version 5.4.3 and later).
> [!NOTE]
> Use the .NET API's [HasFeature()](xref:Yubico.YubiKey.YubiKeyFeatureExtensions.HasFeature%28Yubico.YubiKey.IYubiKeyDevice%2CYubico.YubiKey.YubiKeyFeature%29) method to check if a key has the YubiHSM Auth application.
> Use the .NET
>
Copy link
Collaborator

Choose a reason for hiding this comment

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

Extra line causing formatting issues.


## Available

All YubiKeys with the YubiHSM Auth application (included in firmware version 5.4.3 and later).
> [!NOTE]
> Use the .NET API's [HasFeature()](xref:Yubico.YubiKey.YubiKeyFeatureExtensions.HasFeature%28Yubico.YubiKey.IYubiKeyDevice%2CYubico.YubiKey.YubiKeyFeature%29) method to check if a key has the YubiHSM Auth application.
> Use the .NET
>
Copy link
Collaborator

Choose a reason for hiding this comment

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

Extra line causing formatting issues.

Comment on lines 56 to 75
Add a
credential. | [Try add credential](xref:Yubico.YubiKey.YubiHsmAuth.YubiHsmAuthSession.TryAddCredential(System.ReadOnlyMemory{System.Byte},Yubico.YubiKey.YubiHsmAuth.CredentialWithSecrets,System.Nullable{System.Int32}@)) |
| [Change management key](xref:Yubico.YubiKey.YubiHsmAuth.YubiHsmAuthSession.TryChangeManagementKey(System.ReadOnlyMemory{System.Byte},System.ReadOnlyMemory{System.Byte},System.Nullable{System.Int32}@)) |
Change the management
key. | [Try change management key](xref:Yubico.YubiKey.YubiHsmAuth.YubiHsmAuthSession.TryChangeManagementKey(System.ReadOnlyMemory{System.Byte},System.ReadOnlyMemory{System.Byte},System.Nullable{System.Int32}@)) |
| [Delete credential](xref:Yubico.YubiKey.YubiHsmAuth.YubiHsmAuthSession.DeleteCredential(System.ReadOnlyMemory{System.Byte},System.String)) |
Delete a
credential. | [Try delete credential](xref:Yubico.YubiKey.YubiHsmAuth.YubiHsmAuthSession.TryDeleteCredential(System.ReadOnlyMemory{System.Byte},System.String,System.Nullable{System.Int32}@)) |
| [Get AES-128 session keys](xref:Yubico.YubiKey.YubiHsmAuth.YubiHsmAuthSession.GetAes128SessionKeys(System.String,System.ReadOnlyMemory{System.Byte},System.ReadOnlyMemory{System.Byte},System.ReadOnlyMemory{System.Byte})) |
Calculate session keys from an AES-128 credential. These session keys are used to establish a secure session with a
YubiHSM 2 device. | n/a |
| [Get application version](xref:Yubico.YubiKey.YubiHsmAuth.YubiHsmAuthSession.GetApplicationVersion) | Get the version
of the YubiHSM Auth application returned as a major, minor, and patch value. | n/a |
| [Get management key retries](xref:Yubico.YubiKey.YubiHsmAuth.YubiHsmAuthSession.GetManagementKeyRetries) | Get the
number of retries remaining for the management key. | n/a |
| [List credentials](xref:Yubico.YubiKey.YubiHsmAuth.YubiHsmAuthSession.ListCredentials) | Get the public properties of
all credentials in the YubiHSM Auth application, along with the number of retries remaining for each. | n/a |
| [Reset application](xref:Yubico.YubiKey.YubiHsmAuth.YubiHsmAuthSession.ResetApplication) | Reset the YubiHSM Auth
application, which will delete all credentials and set the management key to its default value (all zeros). | n/a |
Copy link
Collaborator

Choose a reason for hiding this comment

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

Table not rendering correctly.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Addressed all of these now hopefully!

@DennisDyallo DennisDyallo merged commit 38f4b11 into develop Jul 29, 2024
4 checks passed
@DennisDyallo DennisDyallo deleted the dennisdyallo/format-docs branch July 29, 2024 11:45
Copy link

Code Coverage

Package Line Rate Branch Rate Complexity Health
Yubico.Core 42% 31% 4257
Yubico.YubiKey 50% 47% 19089
Summary 49% (31993 / 65943) 44% (8185 / 18519) 23346

Minimum allowed line rate is 40%

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants