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

wrapLatestCommitMsg script failed ignoring a code block #117

Open
tehraninasab opened this issue Jul 5, 2023 · 0 comments · May be fixed by #135
Open

wrapLatestCommitMsg script failed ignoring a code block #117

tehraninasab opened this issue Jul 5, 2023 · 0 comments · May be fixed by #135

Comments

@tehraninasab
Copy link
Contributor

tehraninasab commented Jul 5, 2023

The script doesn't ignore the code block in the following commit message:

scripts/styleChecker.fsx: add

Add styleChecker.fsx script to check style of our F#, TS
and YML codes.

The `git restore package.json` command in the
styleChecker.fsx script was failing with the following
error, even when I was running the
`git config --global --add safe.directory '*'` command in
both the styleChecker.fsx script and in the CI. In the
issue [1], it's suggested by someone to use --system
instead of --global in the mentioned command, when the
git command is running in a container, which solved the
problem.

```
fatal: detected dubious ownership in repository at '/__w/conventions/conventions'
To add an exception for this directory, call:

	git config --global --add safe.directory /__w/conventions/conventions

Error when running 'git restore package.json'
Fsdk.Process+ProcessFailed: Exception of type 'Fsdk.Process+ProcessFailed' was thrown.
   at Fsdk.Process.ProcessResult.Unwrap(String errMsg)
   at Fsdk.Process.ProcessResult.UnwrapDefault()
   at FSI_0002.RunPrettier(String arguments)
   at <StartupCode$FSI_0002>.$FSI_0002.main@()
Stopped due to error
Error: Process completed with exit code 1.
```

[1] https://github.com/actions/checkout/issues/1048
parhamsaremi added a commit to parhamsaremi/conventions that referenced this issue Aug 3, 2023
Add failing test for WrapText function based on [1].

[1] nblockchain#117
parhamsaremi added a commit to parhamsaremi/conventions that referenced this issue Aug 7, 2023
Add failing test for WrapText function based on [1].

[1] nblockchain#117
parhamsaremi added a commit to parhamsaremi/conventions that referenced this issue Aug 8, 2023
Add failing test for WrapText function based on [1].

[1] nblockchain#117
parhamsaremi added a commit to parhamsaremi/conventions that referenced this issue Aug 9, 2023
Split paragraph based on codeblock positions so that codeblocks
with two lines, are not broken into multiple paragraphs.

Fixes nblockchain#117
parhamsaremi added a commit to parhamsaremi/conventions that referenced this issue Aug 9, 2023
Split paragraph based on codeblock positions so that codeblocks
with two lines, are not broken into multiple paragraphs.

Fixes nblockchain#117
parhamsaremi added a commit to parhamsaremi/conventions that referenced this issue Aug 9, 2023
Split paragraph based on codeblock positions so that codeblocks
with two lines, are not broken into multiple paragraphs.

Fixes nblockchain#117
@parhamsaremi parhamsaremi linked a pull request Aug 9, 2023 that will close this issue
parhamsaremi added a commit to parhamsaremi/conventions that referenced this issue Aug 9, 2023
Add failing test for WrapText function based on [1].

[1] nblockchain#117
parhamsaremi added a commit to parhamsaremi/conventions that referenced this issue Aug 9, 2023
Split paragraph based on codeblock positions so that codeblocks
with two lines, are not broken into multiple paragraphs.

Fixes nblockchain#117
parhamsaremi added a commit to parhamsaremi/conventions that referenced this issue Aug 10, 2023
Add failing test for WrapText function based on [1].

[1] nblockchain#117
parhamsaremi added a commit to parhamsaremi/conventions that referenced this issue Aug 10, 2023
Split paragraph based on codeblock positions so that codeblocks
with two lines, are not broken into multiple paragraphs.

Fixes nblockchain#117
parhamsaremi added a commit to parhamsaremi/conventions that referenced this issue Aug 15, 2023
Split paragraph based on codeblock positions so that codeblocks
with two lines, are not broken into multiple paragraphs.

Fixes nblockchain#117
parhamsaremi added a commit to parhamsaremi/conventions that referenced this issue Aug 15, 2023
Split paragraph based on codeblock positions so that codeblocks
with two lines, are not broken into multiple paragraphs.

Fixes nblockchain#117
parhamsaremi added a commit to parhamsaremi/conventions that referenced this issue Aug 15, 2023
Ignore codeblocks while splitting the text into paragraphs
because there might be a codeblock with multiple paragraphs.

Fixes nblockchain#117
parhamsaremi added a commit to parhamsaremi/conventions that referenced this issue Aug 15, 2023
Ignore codeblocks while splitting the text into paragraphs
because there might be a codeblock with multiple paragraphs.

Fixes nblockchain#117
parhamsaremi added a commit to parhamsaremi/conventions that referenced this issue Aug 15, 2023
Ignore codeblocks while splitting the text into paragraphs
because there might be a codeblock with multiple paragraphs.

Fixes nblockchain#117
parhamsaremi added a commit to parhamsaremi/conventions that referenced this issue Aug 16, 2023
Ignore codeblocks while splitting the text into paragraphs
because there might be a codeblock with multiple paragraphs.

Fixes nblockchain#117
parhamsaremi added a commit to parhamsaremi/conventions that referenced this issue Aug 16, 2023
Ignore codeblocks while splitting the text into paragraphs
because there might be a codeblock with multiple paragraphs.

Fixes nblockchain#117
parhamsaremi added a commit to parhamsaremi/conventions that referenced this issue Aug 16, 2023
Ignore codeblocks while splitting the text into paragraphs
because there might be a codeblock with multiple paragraphs.

Fixes nblockchain#117
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 a pull request may close this issue.

1 participant