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

remove unused mgf.py and its test file #194

Merged
merged 1 commit into from
Dec 19, 2023

Conversation

CunliangGeng
Copy link
Member

@CunliangGeng CunliangGeng commented Dec 4, 2023

The mgf.py is not needed any more, since we already have GNPSSpectrumLoader to parse and load MGF file.

@CunliangGeng CunliangGeng force-pushed the 11-27-refactor_Spectrum branch from 7b7fed8 to 87caba4 Compare December 14, 2023 08:33
@CunliangGeng CunliangGeng force-pushed the 12-04-remove_unused_mgf_parser branch from d633acd to 1602df6 Compare December 14, 2023 08:33
@CunliangGeng CunliangGeng force-pushed the 11-27-refactor_Spectrum branch from 87caba4 to f68f2d4 Compare December 14, 2023 10:00
@CunliangGeng CunliangGeng force-pushed the 12-04-remove_unused_mgf_parser branch from 1602df6 to aa98a56 Compare December 14, 2023 10:00
@CunliangGeng CunliangGeng force-pushed the 11-27-refactor_Spectrum branch from f68f2d4 to f2f89ad Compare December 19, 2023 13:21
@CunliangGeng CunliangGeng force-pushed the 12-04-remove_unused_mgf_parser branch from aa98a56 to 8822c40 Compare December 19, 2023 13:21
Copy link
Member Author

CunliangGeng commented Dec 19, 2023

Merge activity

@CunliangGeng CunliangGeng force-pushed the 11-27-refactor_Spectrum branch from f2f89ad to ab84fab Compare December 19, 2023 13:28
Base automatically changed from 11-27-refactor_Spectrum to dev December 19, 2023 13:30
@CunliangGeng CunliangGeng force-pushed the 12-04-remove_unused_mgf_parser branch from 8822c40 to 42e4e4a Compare December 19, 2023 13:31
@CunliangGeng CunliangGeng merged commit 5e97659 into dev Dec 19, 2023
1 check failed
@CunliangGeng CunliangGeng deleted the 12-04-remove_unused_mgf_parser branch December 19, 2023 13:33
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.

2 participants