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

New releases #451

Merged
merged 4 commits into from
Jul 10, 2024
Merged

New releases #451

merged 4 commits into from
Jul 10, 2024

Conversation

hendrikvanantwerpen
Copy link
Collaborator

@hendrikvanantwerpen hendrikvanantwerpen commented Jul 9, 2024

This prepares new releases for all the crates.

Changes:

  • Updated versions, changelogs, and readmes.

  • Some code cleanup changes, nothing functional.

  • Removed the explicit versions for in-repo (i.e., path) dependencies. This had no real benefit. They had to be in sync with the source version of the dependency anyway, and it only made updating versions more work.

Resolves #450.

Includes a bit of code cleanup.
- tree-sitter-stack-graphs-java v0.4.0
- tree-sitter-stack-graphs-javascript v0.2.0
- tree-sitter-stack-graphs-python v0.2.0
- tree-sitter-stack-graphs-typescript v0.3.0
Copy link

github-actions bot commented Jul 9, 2024

Performance Summary

Comparing base f7e8984 with head f715ed1 on typescript_benchmark benchmark. For details see workflow artifacts. Note that performance is tested on the last commits with changes in stack-graphs, not on every commit.

Before
--------------------------------------------------------------------------------
Command:            base/target/release/tree-sitter-stack-graphs-typescript index -D base.sqlite --max-file-time=30 --hide-error-details -- base/data/typescript_benchmark
Massif arguments:   --massif-out-file=perf.out
ms_print arguments: --x=72 --y=12 base-perf-results/perf.out
--------------------------------------------------------------------------------


    GB
2.264^                                               #                        
     |                                               #                        
     |                                              :#                        
     |                                         ::::::#                        
     |                                         :    :#                     :::
     |                                         :    :#                     : :
     |                                       :::    :#                  :::: :
     |                                     ::: :    :#                  :  : :
     |                                     : : :    :#                  :  : :
     |                              @ :  ::: : :    :#        ::    :::::  : :
     |             @@  :@          :@::: : : : :    :#       @:: ::::   :  : :
     |             @ :::@ : ::    ::@::::: : : :    :#      :@::::@ :   :  : :
   0 +----------------------------------------------------------------------->Gi
     0                                                                   77.36
After
--------------------------------------------------------------------------------
Command:            head/target/release/tree-sitter-stack-graphs-typescript index -D head.sqlite --max-file-time=30 --hide-error-details -- head/data/typescript_benchmark
Massif arguments:   --massif-out-file=perf.out
ms_print arguments: --x=72 --y=12 head-perf-results/perf.out
--------------------------------------------------------------------------------


    GB
1.712^                                                @@@#                    
     |                                                @  #                    
     |                                                @  #                    
     |                                                @  #                 :::
     |                                              ::@  #                :: :
     |                                              : @  #                :: :
     |                                              : @  #                :: :
     |                                           :::: @  #                :: :
     |                                       @   :  : @  #       ::    ::::: :
     |                           :         ::@ : :  : @  #      :::  :::  :: :
     |                      :  :::         @:@::::  : @  #      :::@::::  :: :
     |                     @:::::: ::: :   @:@::::  : @  #    :::::@::::  :: :
   0 +----------------------------------------------------------------------->Gi
     0                                                                   85.03


#### Changed

- Failure to index a file will not abort indexing anymore, but simply mark the file as failed, as we already do for files with parse errors.

#### Removed

- The NPM distribution has been deprecated.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nit: I would say ‘disabled’ or ‘removed’ rather than ‘deprecated’, since the latter implies it still works.

@hendrikvanantwerpen hendrikvanantwerpen merged commit d7665d1 into main Jul 10, 2024
16 checks passed
@hendrikvanantwerpen hendrikvanantwerpen deleted the releases branch July 10, 2024 14:54
@Clydo1976
Copy link

releases help plz

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.

Release new version of tree-sitter-stack-graphs-python & tree-sitter-stack-graphs-typescript
3 participants