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

✨ add a nu-check verification CI #771

Merged
merged 30 commits into from
Mar 12, 2024
Merged
Changes from 1 commit
Commits
Show all changes
30 commits
Select commit Hold shift + click to select a range
b1cd54a
:construction_worker: add github CI
AucaCoyan Mar 3, 2024
ce3146c
:bug: fix weird bug of actions/checkout@v4 not working
AucaCoyan Mar 3, 2024
c2b039a
:bug: it was missing a `v`
AucaCoyan Mar 3, 2024
ce5d544
:sparkles: add toolkit and lint fn
AucaCoyan Mar 3, 2024
77aeab0
:fire: remove unused action
AucaCoyan Mar 3, 2024
ecec9c4
Merge branch 'main' into add-basic-ci
AucaCoyan Mar 4, 2024
c84c802
:construction_worker: sample script with `nu-check` and `nu --ide-check`
AucaCoyan Mar 7, 2024
66e8d92
:sparkles: now `toolkit` generates a list of errors on each run
AucaCoyan Mar 8, 2024
617cb41
Merge branch 'main' into add-basic-ci
AucaCoyan Mar 9, 2024
3a16239
:fire: delete `nu-ide-and...` file
AucaCoyan Mar 9, 2024
d1e9ac2
:fire: remove unused code or comments
AucaCoyan Mar 9, 2024
3a6f371
Merge branch 'main' into add-basic-ci
AucaCoyan Mar 10, 2024
c4dda5d
Merge branch 'main' into add-basic-ci
AucaCoyan Mar 10, 2024
19e3d55
:sparkles: add failure if `toolkit check pr` fails
AucaCoyan Mar 10, 2024
5d2cf85
:truck: rename some vars
AucaCoyan Mar 10, 2024
4fe7592
:construction_worker: add features dataframe and extra to the CI
AucaCoyan Mar 11, 2024
e2d23be
:sparkles: fork analysis of all files vs git modified files
AucaCoyan Mar 11, 2024
57ebca9
:sparkles: add `full` flag to check all files
AucaCoyan Mar 11, 2024
14e9881
:sparkles: add "file is ok" line and ending OK line
AucaCoyan Mar 11, 2024
56a6121
:bug: change git diff algorithm
AucaCoyan Mar 11, 2024
0fa7432
Merge branch 'main' into add-basic-ci
AucaCoyan Mar 12, 2024
d20ee20
:bug: change `--name-status` for `name-only`
AucaCoyan Mar 12, 2024
8eba605
:bug: try with files = ""
AucaCoyan Mar 12, 2024
3db99a1
:bug: fix git branch bug
AucaCoyan Mar 12, 2024
f1cfae4
:sparkles: checkout `origin/main` in addition to PR branch
AucaCoyan Mar 12, 2024
1f93670
:sparkles: filter only the `*.nu` files
AucaCoyan Mar 12, 2024
b2d3af7
:adhesive_bandage: tiny fixes
AucaCoyan Mar 12, 2024
d190699
:sparkles: reject span column
AucaCoyan Mar 12, 2024
5d5b81d
:bug: you don't know if you will get the spam column
AucaCoyan Mar 12, 2024
a037b6f
:adhesive_bandage: remove an usused print of git branch
AucaCoyan Mar 12, 2024
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
👷 sample script with nu-check and nu --ide-check
  • Loading branch information
AucaCoyan committed Mar 7, 2024
commit c84c8021a481c038b454160fb61537e7386c1bb6
14 changes: 14 additions & 0 deletions nu-ide-and-nu-check-bug.nu
AucaCoyan marked this conversation as resolved.
Show resolved Hide resolved
Original file line number Diff line number Diff line change
@@ -0,0 +1,14 @@
let dir = (pwd)
let file = [ $dir, "git-aliases.nu"] | path join
print $file

# we download the file from github
http get https://raw.githubusercontent.com/nushell/nu_scripts/25514da84d4249ecebdb74c3a23c7184fcc76f50/aliases/git/git-aliases.nu
| save "git-aliases.nu" --force

let result_nu_check = http get https://raw.githubusercontent.com/nushell/nu_scripts/25514da84d4249ecebdb74c3a23c7184fcc76f50/aliases/git/git-aliases.nu
| nu-check
print $"nu-check says `git-aliases.nu` is: ($result_nu_check)\n"

print "... but nu --ide-check says it has no errors, only hints"
nu --ide-check 10 $"($file)" | to text | ['[', $in, ']'] | str join | from json