Skip to content

[DEV] Merge new-poc with dev #449

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
Merged
Sevhena merged 311 commits into from
Mar 16, 2025
Merged

[DEV] Merge new-poc with dev #449

Sevhena merged 311 commits into from
Mar 16, 2025

Conversation

@Sevhena
Copy link
Collaborator

@Sevhena Sevhena commented Mar 7, 2025

Description

  • Adding all up-to-date code to to dev

How Has This Been Tested?

  • Tests run through GitHub Actions

Ayushi1972 and others added 30 commits November 9, 2024 13:26
@Ayushi1972
fixed silly things
6352bbe
@Ayushi1972
update on output files from last run
db87805
@Sevhena
Merge branch 'poc' of https://github.com/ssm-lab/capstone--source-cod…
bb6aa0b 
…e-optimizer into poc
@Sevhena
fixed refactorer
5817ce5
@Sevhena
#239: Implemented Member Ignoring Method Refactoring
3ac2ae6
@tanveer-brar
Added framework for long parameter list refactoring
795e526
@tanveer-brar
Added refactoring logic for long param list code smell(pending catego…
d466527 
…rization for encapsulated parameters list)
@tanveer-brar
Added long param list function example
6282d2d
@tanveer-brar
Refactorer Code standardarization: refactorer naming, PyLint line num…
4bc6705 
…ber, emission check
@tanveer-brar
Added parameter grouping for Long Parameter List refactoring. Updated…
0e9bb17 
… threshold for PyLint smell detection
@Sevhena
added correct success message to long param list refactor
14169c3
@Ayushi1972
renamed and added refactor for unused variables and class attributes
b04614b
@Ayushi1972
fixed silly issues and added to test case
55b8c4b
@Ayushi1972
returned test case file to before
628dc2f
@Sevhena
made config changes
5ef01b7
@Ayushi1972
added test case for unused imports, variables, and class attributes
8c3c0a3
@Ayushi1972
Merge branch 'poc' of https://github.com/ssm-lab/capstone--source-cod…
966f45e 
…e-optimizer into poc
@Ayushi1972
changed deleting to replace with empty line
955aacc
Long message chain refactorer done
410388b
Merge branch 'poc' of https://github.com/Sevhena/capstone--source-cod…
9f9f00c 
…e-optimizer into poc
@Sevhena
added copy of input test file
d70725f
@Sevhena
fixed lmc refactorer
9c46dc6
added tests for example 2
deae825
Merge branch 'poc' of https://github.com/Sevhena/capstone--source-cod…
2adabea 
…e-optimizer into poc
testing
bc78f6c
@nivethakuruparan
Changed ending message
4fdcfb3
@Ayushi1972
Added custom pylint smell for unused vars
96a9654
@Ayushi1972
Merge branch 'poc' of https://github.com/ssm-lab/capstone--source-cod…
96b406b 
…e-optimizer into poc
@Ayushi1972
fixed test copy for my test cases
182b910
@Ayushi1972
updated custom smell for unused vars to include unused class attributes
7a0d4fd
Sevhena and others added 21 commits March 10, 2025 11:14
@Sevhena
fixed attribute error in MIM refactorer
015adf2
@Sevhena
fixed test structure
f8127af
@tanveer-brar
Fixed failing tests for CodeCarbon Returns #405
9628ef4
@Sevhena
fixed issue where CRC was printing to output_file when overwrite is True
149c292
@Sevhena
removed UVA smell files
511b4fb
@Sevhena
Merge branch 'new-poc' of https://github.com/ssm-lab/capstone--source…
869d233 
…-code-optimizer into new-poc
@Sevhena
fixed CRC detection test bug
8d15900
@Sevhena
removed unused method from LMC refactorer
3133bab
@Sevhena
removed utils tests
2c1c7ce
@nivethakuruparan
Added test for Analyzer Controller (#400)
f48a72f
@nivethakuruparan
Added docstring test for CRC refactorer (#411)
c373512
Added completed benchmarking closes #458
4e8410b
Added completed benchmarking closes #458
624abfc
Merge branch 'new-poc' of https://github.com/Sevhena/capstone--source…
6bdcaa9 
…-code-optimizer into new-poc
@Sevhena
benchmarking fix
dea22ff
@Sevhena
changed handling of benchmarking artifacts
2ceb8f5
@Sevhena
updated worflows to only check non-omitted files
a1c82e6
@Sevhena
ruff lint fixes
521c1ff
@Sevhena
omit logging route from coverage
fc86721
@Ayushi1972
fixed lec multi file refactor test case
4239916
@Ayushi1972
addressed blank whitespaces
ba4e983
@Sevhena Sevhena marked this pull request as ready for review March 16, 2025 18:41
nivethakuruparan
nivethakuruparan approved these changes Mar 16, 2025
View reviewed changes
Copy link
Collaborator

@nivethakuruparan nivethakuruparan left a comment

Choose a reason for hiding this comment

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

LGTM

mmyaaaaa
mmyaaaaa approved these changes Mar 16, 2025
View reviewed changes
Copy link
Contributor

@mmyaaaaa mmyaaaaa left a comment

Choose a reason for hiding this comment

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

i am so excited for this

@Sevhena Sevhena merged commit 1c8eb73 into dev Mar 16, 2025
2 checks passed
@Sevhena Sevhena deleted the new-poc branch March 16, 2025 19:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@nivethakuruparan nivethakuruparan nivethakuruparan approved these changes

@tanveer-brar tanveer-brar Awaiting requested review from tanveer-brar

@Ayushi1972 Ayushi1972 Awaiting requested review from Ayushi1972

+1 more reviewer

@mmyaaaaa mmyaaaaa mmyaaaaa approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

dev Related to the Initial Development

Projects

None yet

Milestone

Rev 1

Development

Successfully merging this pull request may close these issues.

6 participants

@Sevhena @mmyaaaaa @nivethakuruparan @Ayushi1972 @tanveer-brar