Updated `write.configs.ed` to throw an error if `revision` not specified

[meetagrawal09]

Description

This PR fixes #3142

Review Time Estimate

• [ ] Immediately
• [ ] Within one week
• [X] When possible

Types of changes

• [X] Bug fix (non-breaking change which fixes an issue)
• [ ] New feature (non-breaking change which adds functionality)
• [ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

• [ ] My change requires a change to the documentation.
• [ ] My name is in the list of CITATION.cff
• [ ] I have updated the CHANGELOG.md.
• [ ] I have updated the documentation accordingly.
• [X] I have read the CONTRIBUTING document.
• [ ] I have added tests to cover my changes.
• [ ] All new and existing tests passed.

[meetagrawal09]

@mdietze, @Aariq should I turn to using logger.error in-place of logger.severe for the fix ?

[mdietze]

I think either error or severe is fine. We tend to lean towards error over severe in cases where there are multiple ensemble members and/or locations and only a subset might be failing (i.e. we don't want to kill other ensemble members over an error in just one).

If this fix eliminates the dependency on PEcAn.DB then the DESCRIPTION and make should also be updated