Replies: 3 comments 1 reply
-
|
that would be great :) |
Beta Was this translation helpful? Give feedback.
0 replies
-
|
@lnicola Thanks for starting this conversation! I think the biggest problem was the fact that the PR link was indented at 3 spaces, a level that various editors would interpret differently. In your proposal I like that:
Grouping by change type is a bonus improvement. :) |
Beta Was this translation helpful? Give feedback.
0 replies
-
|
@lnicola Any ideas on how we get from here to there? |
Beta Was this translation helpful? Give feedback.
1 reply
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
-
The changelog currently looks like this:
It takes a lot of space in both directions and it feels like half of the PRs that touch it get the indentation wrong.
I propose we switch to a more standard layout, like:
Breaking
TryFrom<&[(&str, &str); N]>for CslStringList withimpl FromIterator<CslStringListEntry> for CslStringList,impl FromIterator<String> for CslStringListandimpl<'a> FromIterator<&'a str> for CslStringListCslStringListIteratorreturns aCslStringListEntryinstead of(String, String)in order to differentiate betweenkey=valueentries andflagsCslStringList::fetch_name_valuereturnsOption<String>instead ofResult<Option<String>>, better reflecting the semantics of the GDAL APISpatialRef::set_axis_mapping_strategynow takes&mut self(Make SpatialRef::set_axis_mapping_strategy take self by mut #461)Dataset::raster_countnow returnsusizeandDataset::rasterbandtakesusizeinstead ofisize(Make raster index usize #434)ExtendedDataTypeno longer implementsClone,PartialEqandEq(Don't derive Clone, PartialEq and Eq for ExtendedDataType #451)LayerIterator,LayerOptionsandTransactionwere moved tocrate::vector(Maintenance cleanup ondataset.rs#447)Added
aspect,color_relief,hillshade,roughness,slope,terrain_ruggedness_index,topographic_position_index(GDAL DEM Processing Routines #456){Display|FromStr}forResampleAlgandResampleAlg::iter(Added{Display|FromStr} for ResampleAlgandResampleAlg::iter. #462)impl Extend<CslStringListEntry> for CslStringListandCslStringList::merge(AddedExtendand variousFromIteratorimplementations toCslStringList. #457)CslStringList::get_field,CslStringList::find_string,CslStringList::partial_find_string,CslStringList::find_string_case_sensitive,CslStringList::into_ptr,CslStringList::add_name_value(Fix/454 #455)MajorObject::gdal_object_ptrandDataset::c_datasetare no longerunsafe(Maintenance cleanup ondataset.rs#447)Fixed
gdal-sysbuild script no longer fails on development GDAL versions (Work around GDAL dev version numbering #439)If there's support for this, I can do the translation myself.
Beta Was this translation helpful? Give feedback.
All reactions