Nabil Freij

Results 815 comments of Nabil Freij

Extra attempt https://github.com/sunpy/sunpy/pull/3728

I believe it comes around every year.

@SKKSaikia You don't need permission to work on an issue. Just be aware that there is a PR open (https://github.com/sunpy/sunpy/pull/2455) that deals with part of this issue.

I would vote for "coords generation" especially with FITS.

> Am I supposed to modify anything else? Looks good to me. This will need more reviewers first however before we merge.

Oh as long as the tests pass as well. I forgot about those. > Ok great. Maybe I should begin working on another issue. are there any suggestions? I have...

> > Oh as long as the tests pass as well. I forgot about those. > > > > > Ok great. Maybe I should begin working on another issue....

If you either rebase the changes ontop of the main branch or merge in main instead, hopefully the CI will pass now.

> > If you either rebase the changes ontop of the main branch or merge in main instead, hopefully the CI will pass now. > > I rebased but I...

I am going to take Alberts changes as an approval of the code changes here.