removed deprecated dependency on archive.m_context

nomad-coe / atomistic-parsers

Apache License 2.0

7 stars 3 forks source link

removed deprecated dependency on archive.m_context #113

Closed Bernadette-Mohr closed 4 months ago

JFRudzinski commented 4 months ago

@ladinesa Can you take a look at this? The installation is suddenly failing in the pipeline. It was working fine when I made a PR last night. The changes in this PR have no connection with this, but just to be sure I made a simple copy of develop and just added a comment to the code, and get the same errors.

ladinesa commented 4 months ago

I think it is because mpcdf is down. maybe we can rerun later.

JFRudzinski commented 4 months ago

I think it is because mpcdf is down. maybe we can rerun later.

oh ok, makes sense...I did not notice this, thanks

coveralls commented 4 months ago

Pull Request Test Coverage Report for Build 9595380425

Details

4 of 4 (100.0%) changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.06%) to 88.201%

Totals
Change from base Build 9593354407:	0.06%
Covered Lines:	6242
Relevant Lines:	7077

💛 - Coveralls

JFRudzinski commented 4 months ago

@Bernadette-Mohr It's fixed! You can go ahead and merge it now