hpack icon indicating copy to clipboard operation
hpack copied to clipboard

Update README.md

Open flip111 opened this issue 6 years ago • 4 comments
trafficstars

add info about extensions

flip111 avatar Dec 27 '18 12:12 flip111

Should this simple PR be merged or closed?

andreasabel avatar Nov 11 '20 10:11 andreasabel

I'm not sure. I think it is a good thing if the Hpack docs are useful on their own. On the other hand, for stuff that is almost literally mapped to .cabal it might make sense to reference the Cabal docs instead. Thoughts?

sol avatar Nov 12 '20 04:11 sol

I'm not sure. I think it is a good thing if the Hpack docs are useful on their own. On the other hand, for stuff that is almost literally mapped to .cabal it might make sense to reference the Cabal docs instead. Thoughts?

Certainly. But the doc in this PR is better than no doc, so it can't hurt.

andreasabel avatar Jan 20 '21 08:01 andreasabel

Problems with a large number of links to the Cabal package description specification are (a) linking to which version of the specification? And (b) the 'cost' of maintaining the links.

Rather that treat these two Hpack keys differently from all the others, I think a better solution is:

  • provide one introductory link in this document to the Cabal User Guide, and point the reader to the relevant part; and
  • state expressly that a . in the Cabal column means that the Hpack key corresponds directly to a field in the Cabal package description of the same name.

mpilgrem avatar Jun 16 '24 20:06 mpilgrem