Settings-view: Many (all?) core packages show "No README"

Created on 3 Aug 2018  Â·  6Comments  Â·  Source: atom/settings-view

_From @jaller94 on August 3, 2018 5:29_

Prerequisites

Description

The settings page of the about package says No README

Steps to Reproduce

  1. Go into the packages settings
  2. Search and select the about package

Expected behavior: The Readme shows a description of the package’s use. Most likely the text would be the same text as this repository’s README.md.

Actual behavior: It says "No README"

Reproduces how often: every time

Versions

Atom 1.28.0 on Linux_x64
Atom 1.29.0 on MacOS

_Copied from original issue: atom/about#85_

triaged

Most helpful comment

@annthurium and I investigated this further and discovered that the re-introduction of ASAR archiving was indeed the culprit. It turns out that we've been excluding Markdown files from the ASAR package which meant that the Settings view could never find README.md files for bundled core packages. PR incoming.

All 6 comments

Thanks for the report @jaller94 - I noticed this was the case with other packages besides atom/about (e.g. archive-view, autocomplete-atom-api, autocomplete-css).

Reproduced on macOS 10.12.6 with 1.31.0-nightly0.

Do they all start with the letter a? 😅

Also affected: autoflow, autosave
I think I found a pattern.. 😉

lol 😄 No it looks like all the ones I've checked so far are showing the same thing including github and tabs for example, so not just a packages.

I tried a series of old Atom releases and this problem seems to have occurred between Atom 1.20.1 and 1.21.0. Looking at the release notes for 1.21.0, it seems that there are two changes that are possible culprits:

I'll continue looking for the root cause.

@annthurium and I investigated this further and discovered that the re-introduction of ASAR archiving was indeed the culprit. It turns out that we've been excluding Markdown files from the ASAR package which meant that the Settings view could never find README.md files for bundled core packages. PR incoming.

Was this page helpful?
0 / 5 - 0 ratings