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.

Sign up for GitHub

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

Jump to bottom

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 •
edited by knocte

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
@parhamsaremi
FileConventions.Tests: add failing test
fd7cfab 
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
@parhamsaremi
FileConventions.Tests: add failing test
d3221a8 
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
@parhamsaremi
FileConventions.Tests: add failing test
4f76c3d 
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
@parhamsaremi
FileConventions: fix WrapText function
795b515 
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
@parhamsaremi
FileConventions: fix WrapText function
9da447a 
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
@parhamsaremi
FileConventions: fix WrapText function
cd1724b 
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
Open
parhamsaremi added a commit to parhamsaremi/conventions that referenced this issue Aug 9, 2023
@parhamsaremi
FileConventions.Tests: add failing test
8898f23 
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
@parhamsaremi
FileConventions: fix WrapText function
5a924f9 
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
@parhamsaremi
FileConventions.Tests: add failing test
1fa1684 
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
@parhamsaremi
FileConventions: fix WrapText function
0614dde 
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
@parhamsaremi
FileConventions: fix WrapText function
bb9ce72 
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
@parhamsaremi
FileConventions: fix WrapText function
17cba0d 
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
@parhamsaremi
FileConventions: fix WrapText function
0e9b157 
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
@parhamsaremi
FileConventions: fix WrapText function
f3486b8 
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
@parhamsaremi
FileConventions: fix WrapText function
0e1b393 
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
@parhamsaremi
FileConventions: fix WrapText function
a69af6f 
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
@parhamsaremi
FileConventions: fix WrapText function
fe739b6 
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
@parhamsaremi
FileConventions: fix WrapText function
fc2c4d5 
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
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

FileConventions: fix WrapText function parhamsaremi/conventions
1 participant
@tehraninasab