Skip to content

Restructuring and implemented make.go#3

Open
soumeh01 wants to merge 2 commits intomainfrom
Restructure
Open

Restructuring and implemented make.go#3
soumeh01 wants to merge 2 commits intomainfrom
Restructure

Conversation

@soumeh01
Copy link
Owner

@soumeh01 soumeh01 commented Sep 9, 2022

The change includes

  • Restructuring

    • All packages are moved under /pkg
    • Entry points are moved under /cmd
  • Implemented a make utility for operations like build, test, etc. It can be used as below
    Usage:
    go run cmd/make/make.go help

@soumeh01 soumeh01 force-pushed the Restructure branch 2 times, most recently from 95d8f5e to 2cd7a88 Compare September 9, 2022 12:13
@github-actions
Copy link

github-actions bot commented Sep 9, 2022

Test Results

       3 files  ±0       21 suites  +3   0s ⏱️ ±0s
1 171 tests  - 2  1 171 ✔️  - 2  0 💤 ±0  0 ±0 
3 513 runs   - 6  3 513 ✔️  - 6  0 💤 ±0  0 ±0 

Results for commit 0beba87. ± Comparison against base commit b9396c2.

This pull request removes 1173 and adds 1171 tests. Note that renamed tests count towards both.
eventlist ‑ Test_includes_Set
eventlist ‑ Test_includes_Set/empty
eventlist ‑ Test_includes_Set/to_empty
eventlist ‑ Test_includes_Set/to_one
eventlist ‑ Test_includes_String
eventlist ‑ Test_includes_String/empty
eventlist ‑ Test_includes_String/nil
eventlist ‑ Test_includes_String/one
eventlist ‑ Test_includes_String/two
eventlist ‑ Test_infoOpt
…
eventlist/cmd/eventlist ‑ Test_includes_Set
eventlist/cmd/eventlist ‑ Test_includes_Set/empty
eventlist/cmd/eventlist ‑ Test_includes_Set/to_empty
eventlist/cmd/eventlist ‑ Test_includes_Set/to_one
eventlist/cmd/eventlist ‑ Test_includes_String
eventlist/cmd/eventlist ‑ Test_includes_String/empty
eventlist/cmd/eventlist ‑ Test_includes_String/nil
eventlist/cmd/eventlist ‑ Test_includes_String/one
eventlist/cmd/eventlist ‑ Test_includes_String/two
eventlist/cmd/eventlist ‑ Test_infoOpt
…

♻️ This comment has been updated with latest results.

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 this pull request may close these issues.

1 participant