telefonicaid / fiware-cygnus

A connector in charge of persisting context data sources into other third-party databases and storage systems, creating a historical view of the context
https://fiware-cygnus.rtfd.io/
GNU Affero General Public License v3.0
64 stars 105 forks source link

[doc] Link to Contribution Guideline page is not valid. #1498

Open vjain641 opened 6 years ago

vjain641 commented 6 years ago

Referred to the Introduction section. Here Community contribution guidelines link mentioned is not valid, opening which it says Maze found. I also checked whether the correct link is provided on stable and latest release but result is same. Could you provide a reference to the Cygnus community contribution guidelines?

fgalan commented 6 years ago

This seems to be the right one: https://github.com/telefonicaid/fiware-cygnus/blob/master/doc/contributing/contributing_guidelines.md

It would be great if somebody is able to fix it (@jason-fox ?) so it also renders correctly at RTD. Thanks!

jason-fox commented 6 years ago

If you run the documentation from the command line, you can see that several pages such as the contributing_guidelines.md are not found within the RTD Documentation. The fix would be to either:

mkdocs serve
INFO    -  Building documentation... 
INFO    -  Cleaning site directory 
WARNING -  The page "index.md" contained a hyperlink to "../doc/contributing/contributing_guidelines.md" which is not listed in the "pages" configuration. 
WARNING -  The page "cygnus-common/installation_and_administration_guide/introduction.md" contained a hyperlink to "contributing/contributing_guidelines.md" which is not listed in the "pages" configuration. 
WARNING -  The page "cygnus-common/installation_and_administration_guide/yafs.md" contained a hyperlink to "cygnus-common/installation_and_administration_guide/runninh_as_process.md" which is not listed in the "pages" configuration. 
WARNING -  The page "cygnus-common/installation_and_administration_guide/yafs.md" contained a hyperlink to "cygnus-common/installation_and_administration_guide/running_as_service.md" which is not listed in the "pages" configuration. 
WARNING -  The page "cygnus-ngsi/installation_and_administration_guide/configuration_examples.md" contained a hyperlink to "cygnus-ngsi/flume_extensions_catalogue/ngsi_mongo.md" which is not listed in the "pages" configuration. 
WARNING -  The page "cygnus-ngsi/installation_and_administration_guide/configuration_examples.md" contained a hyperlink to "cygnus-ngsi/flume_extensions_catalogue/ngsi_sth.md" which is not listed in the "pages" configuration. 
WARNING -  The page "cygnus-ngsi/flume_extensions_catalogue/ngsi_cartodb_sink.md" contained a hyperlink to "ngsi_rest_handler.md" which is not listed in the "pages" configuration. 
WARNING -  The page "cygnus-ngsi/flume_extensions_catalogue/ngsi_ckan_sink.md" contained a hyperlink to "ngsi_rest_handler.md" which is not listed in the "pages" configuration. 
WARNING -  The page "cygnus-ngsi/flume_extensions_catalogue/ngsi_dynamodb_sink.md" contained a hyperlink to "ngsi_rest_handler.md" which is not listed in the "pages" configuration. 
WARNING -  The page "cygnus-ngsi/flume_extensions_catalogue/ngsi_dynamodb_sink.md" contained a hyperlink to "ngsi_rest_handler.md" which is not listed in the "pages" configuration. 
WARNING -  The page "cygnus-ngsi/flume_extensions_catalogue/ngsi_dynamodb_sink.md" contained a hyperlink to "ngsi_grouping_interceptor.md" which is not listed in the "pages" configuration. 
WARNING -  The page "cygnus-ngsi/flume_extensions_catalogue/ngsi_kafka_sink.md" contained a hyperlink to "ngsi_rest_handler.md" which is not listed in the "pages" configuration. 
WARNING -  The page "cygnus-ngsi/flume_extensions_catalogue/ngsi_hdfs_sink.md" contained a hyperlink to "ngsi_rest_handler.md" which is not listed in the "pages" configuration. 
WARNING -  The page "cygnus-ngsi/flume_extensions_catalogue/ngsi_mongo_sink.md" contained a hyperlink to "ngsi_rest_handler.md" which is not listed in the "pages" configuration. 
WARNING -  The page "cygnus-ngsi/flume_extensions_catalogue/ngsi_mysql_sink.md" contained a hyperlink to "ngsi_rest_handler.md" which is not listed in the "pages" configuration. 
WARNING -  The page "cygnus-ngsi/flume_extensions_catalogue/ngsi_postgresql_sink.md" contained a hyperlink to "ngsi_rest_handler.md" which is not listed in the "pages" configuration. 
WARNING -  The page "cygnus-ngsi/flume_extensions_catalogue/ngsi_sth_sink.md" contained a hyperlink to "ngsi_rest_handler.md" which is not listed in the "pages" configuration. 
WARNING -  The page "cygnus-ngsi/flume_extensions_catalogue/ngsi_test_sink.md" contained a hyperlink to "ngsi_rest_handler.md" which is not listed in the "pages" configuration. 
WARNING -  The page "cygnus-ngsi/flume_extensions_catalogue/ngsi_grouping_interceptor.md" contained a hyperlink to "cygnus-ngsi/flume_extensions_catalogue/deprecated_and_removed.md" which is not listed in the "pages" configuration. 
WARNING -  The page "cygnus-twitter/quick_start_guide.md" contained a hyperlink to "cygnus-twitter/user_and_programmer_guide/README.md" which is not listed in the "pages" configuration. 
fgalan commented 6 years ago

Should contribution guidleines be included in RTD manual? I tend to think in RTD as documentation for "final users" of the component...

In fact, in the Orion repository Contribution Guidelines and Development Manual are not included in RTD.

jason-fox commented 6 years ago

Additional subjects such as Contribution Guidelines, testing policies, code of conduct have their place (and should be found within the docs directory IMHO, but shouldn't be indexed as part of RTD styled documentation. It makes the documentation more difficult to read.

BTW GitHub have a Contribution Guidelines Policy - we should follow it - the file should be called CONTRIBUTING.MD and be placed in root or \docs