Update Main #68

c-git · 2024-01-24T14:28:08Z

Main has fallen out of sync with Develop bringing it back up to date in preparation for publishing

Add README and .gitignore files

Prefer specific version unless testing that code works with old version

Simplify return types for fallible functions

Move main logic into a module called core to make it clear which parts are executing code

Was getting a warning from crates plugin in vscode saying > The version requirement of the target dependency I couldn't figure out what it meant but keeping it at 0.4.19 wasn't providing any additional value.

Provide more info for debugging and better error messages when things go wrong

Getting quite a few if using debug rn so going to change those in this module to info. Side stepping the problem for now. Can be filtered out instead.

To side step problem of other creates logs showing up

Based on example https://docs.rs/clap/latest/clap/_derive/_cookbook/cargo_example_derive/

Test had to be updated as top of parser had changed when moving to proper cargo subcommand

Support VecVecString

- Needed for new way of checking if type is a String type

Ensure code also compiles in release mode

Develop was only there for testing

Release Compile Fix

Add contribution guide

…ion-guide Fix typo in CONTRIBUTING.md

Needed for today's problem https://leetcode.com/problems/pseudo-palindromic-paths-in-a-binary-tree

nassersaazi and others added 30 commits June 11, 2023 02:47

Add README and .gitignore files

e297e89

Merge pull request #11 from nassersaazi/develop

423407a

Add README and .gitignore files

Move code into lib

fedf83a

Put dependencies in lexicographical order

65f8228

Remove white space

45d869f

Set specific serde version

1c0e7c2

Prefer specific version unless testing that code works with old version

Change return type to anyhow::Result

d69d5a4

Simplify return types for fallible functions

Add clap

0152387

Restructure to put executed code into lib

1d735ed

Move main logic into a module called core to make it clear which parts are executing code

Add basic logging support

2fc4eac

Change version of log to stop warning

822d57e

Was getting a warning from crates plugin in vscode saying > The version requirement of the target dependency I couldn't figure out what it meant but keeping it at 0.4.19 wasn't providing any additional value.

Add description

09c62a4

Improve logging and error messages

25524cf

Provide more info for debugging and better error messages when things go wrong

Add check to ensure correct folder

d8f72de

Change return types to anyhow

bbe56ec

Connect up existing code

3cb91b7

Move leetcode url into config

0edfaef

Remove parts of paths that look like they should change

24b77ac

Move GraphQL link into config

9bc6e1e

Add TODOs to document what needs to be done

ddc9af7

Convert from url to slug

64746e5

Add where logs come from

26a0370

Getting quite a few if using debug rn so going to change those in this module to info. Side stepping the problem for now. Can be filtered out instead.

Change all log messages to info

1260810

To side step problem of other creates logs showing up

Fix path missed

8fa8680

Add logging to functions

9c64407

Enable writing out files

9b085d7

Add install instructions to readme

a3b5826

Update CLI to work with cargo properly

385ab3b

Based on example https://docs.rs/clap/latest/clap/_derive/_cookbook/cargo_example_derive/

Update test

84c56dc

Test had to be updated as top of parser had changed when moving to proper cargo subcommand

Add screenshots

30d3ebf

c-git and others added 27 commits July 22, 2023 00:16

Set CI to test all features

8c0f33a

Merge pull request #48 from rust-practice/support_vec_vec_string

b89ddae

Support VecVecString

Stop unused import when compiling in release mode

eff2cb5

Add PartialEq to release builds

e0a627f

- Needed for new way of checking if type is a String type

Test new workflow for release compilation

7bc18b2

Ensure code also compiles in release mode

Switch to only run on main

664c838

Develop was only there for testing

Merge pull request #53 from rust-practice/release_compile_fix

179e2cf

Release Compile Fix

Add contribution guide

4970d62

Wrap long lines using rustfmt with nightly

101b9e2

Remove unneeded comment

7692b7f

Fix typo

c540a24

Replace this to improve readability

4e7ee15

Make updates to contribution guide in preparation for publishing

885d92a

Ask contributors to use develop instead of main

e22474f

Remove cargo check as it's just duplicated work

b796e79

use dprint to format files

707c623

Remove misleading line as it doesn't actually work

e26155f

FIx lints from new version of rust

9166c37

Merge branch 'develop' into contribution-guide

3a0cf3f

Clarify statement in readme

8f4853a

Merge pull request #55 from nassersaazi/contribution-guide

239ed45

Add contribution guide

Update CONTRIBUTING.md

6775ba5

Merge pull request #67 from rust-practice/c-git-fix-typo-in-contribut…

eb49761

…ion-guide Fix typo in CONTRIBUTING.md

Allow spaces after function name

ca43b30

Needed for today's problem https://leetcode.com/problems/pseudo-palindromic-paths-in-a-binary-tree

Add TODO to change how constant is validated

ac7ec63

Version Bump

4257a47

Increasing version number given the number of changes

20adebd

c-git requested a review from AliciaY-720 January 24, 2024 14:28

AliciaY-720 approved these changes Jan 27, 2024

View reviewed changes

c-git merged commit cad01be into main Jan 27, 2024
7 checks passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Update Main #68

Update Main #68

c-git commented Jan 24, 2024 •

edited

Loading

Update Main #68

Update Main #68

Conversation

c-git commented Jan 24, 2024 • edited Loading

c-git commented Jan 24, 2024 •

edited

Loading